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

Commit 776d4ec4 authored by Flare Chiu's avatar Flare Chiu
Browse files

Revert "Run unlock animation on MODE_ONLY_WAKE" 

This reverts commit 5481291d.

Reason for revert: DroidMonitor: Potential culprit for Bug 233958841 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Change-Id: I4f28fac9784da045866bb592c163f834c0dc856b
parent 5481291d

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);