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

Commit f22146d7 authored by Flare Chiu's avatar Flare Chiu Committed by Automerger Merge Worker
Browse files

Merge "Revert "Run unlock animation on MODE_ONLY_WAKE"" into tm-dev am: 39f79b0f 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18610066



Change-Id: Id6cc4135336d22ac4647e990f1a0498874a29d73
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 8debce0d 39f79b0f

packages/SystemUI/src/com/android/systemui/keyguard/KeyguardViewMediator.java

+3 −6
Original line number Diff line number Diff line
@@ -2816,12 +2816,9 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable, 
        }

    }



    /**

     * Receive a wake event from outside this class (most likely bio auth).

     */

    public void onWake(boolean withUnlock) {

        Trace.beginSection("KeyguardViewMediator#onWake");

        mWakeAndUnlocking = withUnlock;

    public void onWakeAndUnlocking() {

        Trace.beginSection("KeyguardViewMediator#onWakeAndUnlocking");

        mWakeAndUnlocking = true;



        keyguardDone();

        Trace.endSection();

packages/SystemUI/src/com/android/systemui/statusbar/phone/BiometricUnlockController.java

+1 −3
Original line number Diff line number Diff line
@@ -500,12 +500,10 @@ public class BiometricUnlockController extends KeyguardUpdateMonitorCallback imp 
                    mUpdateMonitor.awakenFromDream();

                }

                mNotificationShadeWindowController.setNotificationShadeFocusable(false);

                mKeyguardViewMediator.onWake(true /* withUnlock */);

                mKeyguardViewMediator.onWakeAndUnlocking();

                Trace.endSection();

                break;

            case MODE_ONLY_WAKE:

                mKeyguardViewMediator.onWake(false /* withUnlock */);

                break;

            case MODE_NONE:

                break;

        }

packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/BiometricsUnlockControllerTest.java

+1 −18
Original line number Diff line number Diff line
@@ -180,28 +180,11 @@ public class BiometricsUnlockControllerTest extends SysuiTestCase { 
        mBiometricUnlockController.onBiometricAuthenticated(UserHandle.USER_CURRENT,

                BiometricSourceType.FINGERPRINT, true /* isStrongBiometric */);



        verify(mKeyguardViewMediator).onWake(true /* withUnlock */);

        verify(mKeyguardViewMediator).onWakeAndUnlocking();

        assertThat(mBiometricUnlockController.getMode())

                .isEqualTo(BiometricUnlockController.MODE_WAKE_AND_UNLOCK_PULSING);

    }



    @Test

    public void onBiometricAuthenticated_whenFingerprintAndInteractive_wakeWithoutUnlock() {

        reset(mUpdateMonitor);

        reset(mStatusBarKeyguardViewManager);

        when(mStatusBarKeyguardViewManager.isShowing()).thenReturn(false);

        when(mUpdateMonitor.isUnlockingWithBiometricAllowed(anyBoolean())).thenReturn(true);

        when(mDozeScrimController.isPulsing()).thenReturn(false);

        // the value of isStrongBiometric doesn't matter here since we only care about the returned

        // value of isUnlockingWithBiometricAllowed()

        mBiometricUnlockController.onBiometricAuthenticated(UserHandle.USER_CURRENT,

                BiometricSourceType.FINGERPRINT, true /* isStrongBiometric */);



        verify(mKeyguardViewMediator).onWake(false /* withUnlock */);

        assertThat(mBiometricUnlockController.getMode())

                .isEqualTo(BiometricUnlockController.MODE_ONLY_WAKE);

    }



    @Test

    public void onBiometricAuthenticated_whenFingerprint_notifyKeyguardAuthenticated() {

        when(mUpdateMonitor.isUnlockingWithBiometricAllowed(anyBoolean())).thenReturn(true);