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

Commit 755ad661 authored by Andreas Miko's avatar Andreas Miko Committed by Automerger Merge Worker
Browse files

Merge "Show bouncer after user switch" into udc-dev am: a6b13ba0 

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



Change-Id: Id2ff5261e79c787a7a9b1f0f586038faea26648c
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 44707595 a6b13ba0

packages/SystemUI/src/com/android/keyguard/KeyguardSecurityContainerController.java

+0 −1
Original line number Diff line number Diff line
@@ -531,7 +531,6 @@ public class KeyguardSecurityContainerController extends ViewController<Keyguard 
    public void setOnDismissAction(ActivityStarter.OnDismissAction action, Runnable cancelAction) {

        if (mCancelAction != null) {

            mCancelAction.run();

            mCancelAction = null;

        }

        mDismissAction = action;

        mCancelAction = cancelAction;

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

+24 −22
Original line number Diff line number Diff line
@@ -53,7 +53,6 @@ import android.content.Context; 
import android.content.Intent;

import android.content.IntentFilter;

import android.content.pm.PackageManager.NameNotFoundException;

import android.content.pm.UserInfo;

import android.graphics.Matrix;

import android.hardware.biometrics.BiometricSourceType;

import android.media.AudioAttributes;
@@ -560,17 +559,9 @@ public class KeyguardViewMediator implements CoreStartable, Dumpable, 
        @Override

        public void onUserSwitching(int userId) {

            if (DEBUG) Log.d(TAG, String.format("onUserSwitching %d", userId));

            // Note that the mLockPatternUtils user has already been updated from setCurrentUser.

            // We need to force a reset of the views, since lockNow (called by

            // ActivityManagerService) will not reconstruct the keyguard if it is already showing.

            synchronized (KeyguardViewMediator.this) {

                resetKeyguardDonePendingLocked();

                if (mLockPatternUtils.isLockScreenDisabled(userId)) {

                    // If we are switching to a user that has keyguard disabled, dismiss keyguard.

                dismiss(null /* callback */, null /* message */);

                } else {

                    resetStateLocked();

                }

                adjustStatusBarLocked();

            }

        }
@@ -578,16 +569,9 @@ public class KeyguardViewMediator implements CoreStartable, Dumpable, 
        @Override

        public void onUserSwitchComplete(int userId) {

            if (DEBUG) Log.d(TAG, String.format("onUserSwitchComplete %d", userId));

            if (userId != UserHandle.USER_SYSTEM) {

                UserInfo info = UserManager.get(mContext).getUserInfo(userId);

                // Don't try to dismiss if the user has Pin/Pattern/Password set

                if (info == null || mLockPatternUtils.isSecure(userId)) {

                    return;

                } else if (info.isGuest() || info.isDemo()) {

                    // If we just switched to a guest, try to dismiss keyguard.

                    dismiss(null /* callback */, null /* message */);

                }

            }

            // We are calling dismiss again and with a delay as there are race conditions

            // in some scenarios caused by async layout listeners

            new Handler().postDelayed(() -> dismiss(null /* callback */, null /* message */), 500);

        }



        @Override
@@ -2213,58 +2197,72 @@ public class KeyguardViewMediator implements CoreStartable, Dumpable, 
    private Handler mHandler = new Handler(Looper.myLooper(), null, true /*async*/) {

        @Override

        public void handleMessage(Message msg) {

            String message = "";

            switch (msg.what) {

                case SHOW:

                    message = "SHOW";

                    handleShow((Bundle) msg.obj);

                    break;

                case HIDE:

                    message = "HIDE";

                    handleHide();

                    break;

                case RESET:

                    message = "RESET";

                    handleReset();

                    break;

                case VERIFY_UNLOCK:

                    message = "VERIFY_UNLOCK";

                    Trace.beginSection("KeyguardViewMediator#handleMessage VERIFY_UNLOCK");

                    handleVerifyUnlock();

                    Trace.endSection();

                    break;

                case NOTIFY_STARTED_GOING_TO_SLEEP:

                    message = "NOTIFY_STARTED_GOING_TO_SLEEP";

                    handleNotifyStartedGoingToSleep();

                    break;

                case NOTIFY_FINISHED_GOING_TO_SLEEP:

                    message = "NOTIFY_FINISHED_GOING_TO_SLEEP";

                    handleNotifyFinishedGoingToSleep();

                    break;

                case NOTIFY_STARTED_WAKING_UP:

                    message = "NOTIFY_STARTED_WAKING_UP";

                    Trace.beginSection(

                            "KeyguardViewMediator#handleMessage NOTIFY_STARTED_WAKING_UP");

                    handleNotifyStartedWakingUp();

                    Trace.endSection();

                    break;

                case KEYGUARD_DONE:

                    message = "KEYGUARD_DONE";

                    Trace.beginSection("KeyguardViewMediator#handleMessage KEYGUARD_DONE");

                    handleKeyguardDone();

                    Trace.endSection();

                    break;

                case KEYGUARD_DONE_DRAWING:

                    message = "KEYGUARD_DONE_DRAWING";

                    Trace.beginSection("KeyguardViewMediator#handleMessage KEYGUARD_DONE_DRAWING");

                    handleKeyguardDoneDrawing();

                    Trace.endSection();

                    break;

                case SET_OCCLUDED:

                    message = "SET_OCCLUDED";

                    Trace.beginSection("KeyguardViewMediator#handleMessage SET_OCCLUDED");

                    handleSetOccluded(msg.arg1 != 0, msg.arg2 != 0);

                    Trace.endSection();

                    break;

                case KEYGUARD_TIMEOUT:

                    message = "KEYGUARD_TIMEOUT";

                    synchronized (KeyguardViewMediator.this) {

                        doKeyguardLocked((Bundle) msg.obj);

                    }

                    break;

                case DISMISS:

                    final DismissMessage message = (DismissMessage) msg.obj;

                    handleDismiss(message.getCallback(), message.getMessage());

                    message = "DISMISS";

                    final DismissMessage dismissMsg = (DismissMessage) msg.obj;

                    handleDismiss(dismissMsg.getCallback(), dismissMsg.getMessage());

                    break;

                case START_KEYGUARD_EXIT_ANIM:

                    message = "START_KEYGUARD_EXIT_ANIM";

                    Trace.beginSection(

                            "KeyguardViewMediator#handleMessage START_KEYGUARD_EXIT_ANIM");

                    synchronized (KeyguardViewMediator.this) {
@@ -2282,21 +2280,25 @@ public class KeyguardViewMediator implements CoreStartable, Dumpable, 
                    Trace.endSection();

                    break;

                case CANCEL_KEYGUARD_EXIT_ANIM:

                    message = "CANCEL_KEYGUARD_EXIT_ANIM";

                    Trace.beginSection(

                            "KeyguardViewMediator#handleMessage CANCEL_KEYGUARD_EXIT_ANIM");

                    handleCancelKeyguardExitAnimation();

                    Trace.endSection();

                    break;

                case KEYGUARD_DONE_PENDING_TIMEOUT:

                    message = "KEYGUARD_DONE_PENDING_TIMEOUT";

                    Trace.beginSection("KeyguardViewMediator#handleMessage"

                            + " KEYGUARD_DONE_PENDING_TIMEOUT");

                    Log.w(TAG, "Timeout while waiting for activity drawn!");

                    Trace.endSection();

                    break;

                case SYSTEM_READY:

                    message = "SYSTEM_READY";

                    handleSystemReady();

                    break;

            }

            Log.d(TAG, "KeyguardViewMediator queue processing message: " + message);

        }

    };

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

+25 −10
Original line number Diff line number Diff line
@@ -69,6 +69,7 @@ import com.android.systemui.navigationbar.NavigationBarView; 
import com.android.systemui.navigationbar.NavigationModeController;

import com.android.systemui.navigationbar.TaskbarDelegate;

import com.android.systemui.plugins.statusbar.StatusBarStateController;

import com.android.systemui.settings.UserTracker;

import com.android.systemui.shade.NotificationPanelViewController;

import com.android.systemui.shade.ShadeController;

import com.android.systemui.shade.ShadeExpansionChangeEvent;
@@ -285,8 +286,11 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
    private final boolean mUdfpsNewTouchDetectionEnabled;

    private final UdfpsOverlayInteractor mUdfpsOverlayInteractor;



    private OnDismissAction mAfterKeyguardGoneAction;

    private Runnable mKeyguardGoneCancelAction;

    @VisibleForTesting

    OnDismissAction mAfterKeyguardGoneAction;



    @VisibleForTesting

    Runnable mKeyguardGoneCancelAction;

    private boolean mDismissActionWillAnimateOnKeyguard;

    private final ArrayList<Runnable> mAfterKeyguardGoneRunnables = new ArrayList<>();
@@ -302,6 +306,8 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
    @Nullable private KeyguardBypassController mBypassController;

    @Nullable private OccludingAppBiometricUI mOccludingAppBiometricUI;



    private UserTracker mUserTracker;



    @Nullable private TaskbarDelegate mTaskbarDelegate;

    private final KeyguardUpdateMonitorCallback mUpdateMonitorCallback =

            new KeyguardUpdateMonitorCallback() {
@@ -339,7 +345,8 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
            PrimaryBouncerInteractor primaryBouncerInteractor,

            BouncerView primaryBouncerView,

            AlternateBouncerInteractor alternateBouncerInteractor,

            UdfpsOverlayInteractor udfpsOverlayInteractor

            UdfpsOverlayInteractor udfpsOverlayInteractor,

            UserTracker userTracker

    ) {

        mContext = context;

        mViewMediatorCallback = callback;
@@ -367,6 +374,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
                featureFlags.isEnabled(Flags.WM_ENABLE_PREDICTIVE_BACK_BOUNCER_ANIM);

        mUdfpsNewTouchDetectionEnabled = featureFlags.isEnabled(Flags.UDFPS_NEW_TOUCH_DETECTION);

        mUdfpsOverlayInteractor = udfpsOverlayInteractor;

        mUserTracker = userTracker;

    }



    @Override
@@ -662,14 +670,21 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb 
                if (afterKeyguardGone) {

                    // we'll handle the dismiss action after keyguard is gone, so just show the

                    // bouncer

                    mPrimaryBouncerInteractor.show(/* isScrimmed= */true);

                    if (mLockPatternUtils.isSecure(mUserTracker.getUserId())) {

                        mPrimaryBouncerInteractor.show(true);

                    }

                } else {

                    if (mLockPatternUtils.isSecure(mUserTracker.getUserId())) {

                        // after authentication success, run dismiss action with the option to defer

                        // hiding the keyguard based on the return value of the OnDismissAction

                        mPrimaryBouncerInteractor.setDismissAction(

                                mAfterKeyguardGoneAction, mKeyguardGoneCancelAction);

                    mPrimaryBouncerInteractor.show(/* isScrimmed= */true);

                    // bouncer will handle the dismiss action, so we no longer need to track it here

                        mPrimaryBouncerInteractor.show(true);

                    } else {

                        if (mAfterKeyguardGoneAction != null) {

                            mAfterKeyguardGoneAction.onDismiss();

                        }

                    }

                    mAfterKeyguardGoneAction = null;

                    mKeyguardGoneCancelAction = null;

                }

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

+27 −3
Original line number Diff line number Diff line
@@ -20,6 +20,7 @@ import static com.android.systemui.keyguard.shared.constants.KeyguardBouncerCons 
import static com.android.systemui.keyguard.shared.constants.KeyguardBouncerConstants.EXPANSION_VISIBLE;



import static org.junit.Assert.assertFalse;

import static org.junit.Assert.assertNull;

import static org.junit.Assert.assertTrue;

import static org.mockito.ArgumentMatchers.any;

import static org.mockito.ArgumentMatchers.anyBoolean;
@@ -73,6 +74,7 @@ import com.android.systemui.keyguard.domain.interactor.PrimaryBouncerInteractor; 
import com.android.systemui.navigationbar.NavigationModeController;

import com.android.systemui.navigationbar.TaskbarDelegate;

import com.android.systemui.plugins.ActivityStarter.OnDismissAction;

import com.android.systemui.settings.UserTracker;

import com.android.systemui.shade.NotificationPanelViewController;

import com.android.systemui.shade.NotificationShadeWindowView;

import com.android.systemui.shade.ShadeController;
@@ -192,7 +194,8 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase { 
                        mPrimaryBouncerInteractor,

                        mBouncerView,

                        mAlternateBouncerInteractor,

                        mUdfpsOverlayInteractor) {

                        mUdfpsOverlayInteractor,

                        mock(UserTracker.class)) {

                    @Override

                    public ViewRootImpl getViewRootImpl() {

                        return mViewRootImpl;
@@ -216,14 +219,34 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase { 
    }



    @Test

    public void dismissWithAction_AfterKeyguardGoneSetToFalse() {

    public void dismissWithAction_AfterKeyguardGoneSetToFalse_DismissAction_Set() {

        OnDismissAction action = () -> false;

        Runnable cancelAction = () -> {

        };

        when(mLockPatternUtils.isSecure(anyInt())).thenReturn(true);



        mStatusBarKeyguardViewManager.dismissWithAction(

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



        verify(mPrimaryBouncerInteractor).setDismissAction(eq(action), eq(cancelAction));

        verify(mPrimaryBouncerInteractor).show(eq(true));

        assertNull(mStatusBarKeyguardViewManager.mAfterKeyguardGoneAction);

        assertNull(mStatusBarKeyguardViewManager.mKeyguardGoneCancelAction);

    }



    @Test

    public void dismissWithAction_AfterKeyguardGoneSetToFalse_DismissAction_Called() {

        OnDismissAction action = mock(OnDismissAction.class);

        Runnable cancelAction = () -> {

        };

        when(mLockPatternUtils.isSecure(anyInt())).thenReturn(false);



        mStatusBarKeyguardViewManager.dismissWithAction(

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



        verify(action).onDismiss();

        assertNull(mStatusBarKeyguardViewManager.mAfterKeyguardGoneAction);

        assertNull(mStatusBarKeyguardViewManager.mKeyguardGoneCancelAction);

    }



    @Test
@@ -680,7 +703,8 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase { 
                        mPrimaryBouncerInteractor,

                        mBouncerView,

                        mAlternateBouncerInteractor,

                        mUdfpsOverlayInteractor) {

                        mUdfpsOverlayInteractor,

                        mock(UserTracker.class)) {

                    @Override

                    public ViewRootImpl getViewRootImpl() {

                        return mViewRootImpl;