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

Commit c503f722 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge changes I0ababa6b,I1a1ad69b into tm-dev am: 27a45d8e am: 98a99533 

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



Change-Id: I3b7f0f3c0141424bf6df2c5d15f3e7a29ca7ab76
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 119474c6 98a99533

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

+0 −5
Original line number Diff line number Diff line
@@ -3505,11 +3505,6 @@ public class CentralSurfacesImpl extends CoreStartable implements 


    @Override

    public void onTrackingStopped(boolean expand) {

        if (mState == StatusBarState.KEYGUARD || mState == StatusBarState.SHADE_LOCKED) {

            if (!expand && !mKeyguardStateController.canDismissLockScreen()) {

                mStatusBarKeyguardViewManager.showBouncer(false /* scrimmed */);

            }

        }

    }



    // TODO: Figure out way to remove these.

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

+1 −1
Original line number Diff line number Diff line
@@ -479,12 +479,12 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
        if (mBouncer == null) {

            return;

        }

        mBouncer.hide(destroyView);

        if (mShowing) {

            // If we were showing the bouncer and then aborting, we need to also clear out any

            // potential actions unless we actually unlocked.

            cancelPostAuthActions();

        }

        mBouncer.hide(destroyView);

        cancelPendingWakeupAction();

    }

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

+15 −0
Original line number Diff line number Diff line
@@ -312,12 +312,27 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase { 
        mStatusBarKeyguardViewManager.dismissWithAction(

                action, cancelAction, true /* afterKeyguardGone */);



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

        mStatusBarKeyguardViewManager.hideBouncer(true);

        mStatusBarKeyguardViewManager.hide(0, 30);

        verify(action, never()).onDismiss();

        verify(cancelAction).run();

    }



    @Test

    public void testHidingBouncer_cancelsGoneRunnable() {

        OnDismissAction action = mock(OnDismissAction.class);

        Runnable cancelAction = mock(Runnable.class);

        mStatusBarKeyguardViewManager.dismissWithAction(

                action, cancelAction, true /* afterKeyguardGone */);



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

        mStatusBarKeyguardViewManager.hideBouncer(true);



        verify(action, never()).onDismiss();

        verify(cancelAction).run();

    }



    @Test

    public void testHiding_doesntCancelWhenShowing() {

        OnDismissAction action = mock(OnDismissAction.class);