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

Commit 14436fc6 authored by jiewenlei's avatar jiewenlei Committed by Jiewen Lei
Browse files

Properly log FillDialogReadyToShowMs in cases where IME is not 

   showing.

Test: atest CtsAutoFillServiceTestCases
Flag: EXEMPT minor logging bugfix.
Bug: 399177967
Change-Id: Ib2211de088e1706b48b0277b17ff7d8523b2484a
parent b6d8dcd2

services/autofill/java/com/android/server/autofill/Session.java

+3 −3
Original line number Diff line number Diff line
@@ -5805,6 +5805,9 @@ final class Session 
        }



        synchronized (mLock) {

            long currentTimestampMs = SystemClock.elapsedRealtime();

            mPresentationStatsEventLogger.maybeSetFillDialogReadyToShowMs(

                    currentTimestampMs);

            if (mLastFillDialogTriggerIds == null

                    || !ArrayUtils.contains(mLastFillDialogTriggerIds, filledId)) {

                // Last fill dialog triggered ids are changed.
@@ -5815,7 +5818,6 @@ final class Session 
            }



            if (mImproveFillDialogEnabled && mInlineSessionController.isImeShowing()) {

                long currentTimestampMs = SystemClock.elapsedRealtime();

                long durationMs = currentTimestampMs - mLastInputStartTime;

                if (sVerbose) {

                    Log.d(TAG, "IME is showing. Checking for elapsed time ");
@@ -5848,8 +5850,6 @@ final class Session 
                // max of start input time or the ime finish time

                long effectiveDuration = currentTimestampMs

                        - Math.max(mLastInputStartTime, mImeAnimationFinishTimeMs);

                mPresentationStatsEventLogger.maybeSetFillDialogReadyToShowMs(

                        currentTimestampMs);

                mPresentationStatsEventLogger.maybeSetImeAnimationFinishMs(

                        Math.max(mLastInputStartTime, mImeAnimationFinishTimeMs));

                if (effectiveDuration >= mFillDialogTimeoutMs) {