Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c0f877a7 authored by Joanne's avatar Joanne
Browse files

Do not send event if the buffer is empty. 

Found many logs in bugreport that call sendEvents() even the buffer
is empty. We should avoid unnecessary API call to reduce ipc spam.
This change doesn't fix the potential ipc spam issue, only mitigates
the issue.

Bug: 231313913
Test: manual. Local add log and see the empty api call is skipped
Change-Id: I9c8afc5ae2d06074900886cf5743b1ec61ccb973
parent b2941816

core/java/android/view/contentcapture/MainContentCaptureSession.java

+6 −1
Original line number Diff line number Diff line
@@ -528,7 +528,12 @@ public final class MainContentCaptureSession extends ContentCaptureSession { 
    @Override

    @UiThread

    void flush(@FlushReason int reason) {

        if (mEvents == null) return;

        if (mEvents == null || mEvents.size() == 0) {

            if (sVerbose) {

                Log.v(TAG, "Don't flush for empty event buffer.");

            }

            return;

        }



        if (mDisabled.get()) {

            Log.e(TAG, "handleForceFlush(" + getDebugState(reason) + "): should not be when "