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

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

Merge "Replace StatusBarKeyguardViewManager with KeyguardViewController" into... 

Merge "Replace StatusBarKeyguardViewManager with KeyguardViewController" into tm-dev am: dc25f23c am: d44c1557 am: 0273ea21 am: 5789ab23

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



Change-Id: I31df21d5f4722e0030b0442019e14ac00933fff2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents e104b608 5789ab23

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

+4 −5
Original line number Diff line number Diff line
@@ -28,7 +28,6 @@ import com.android.keyguard.KeyguardSecurityModel.SecurityMode; 
import com.android.systemui.R;

import com.android.systemui.classifier.FalsingCollector;

import com.android.systemui.dagger.qualifiers.Main;

import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager;

import com.android.systemui.statusbar.policy.DevicePostureController;

import com.android.systemui.util.ViewController;

import com.android.systemui.util.concurrency.DelayableExecutor;
@@ -169,7 +168,7 @@ public abstract class KeyguardInputViewController<T extends KeyguardInputView> 
        private final EmergencyButtonController.Factory mEmergencyButtonControllerFactory;

        private final FalsingCollector mFalsingCollector;

        private final DevicePostureController mDevicePostureController;

        private final StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;

        private final KeyguardViewController mKeyguardViewController;



        @Inject

        public Factory(KeyguardUpdateMonitor keyguardUpdateMonitor,
@@ -181,7 +180,7 @@ public abstract class KeyguardInputViewController<T extends KeyguardInputView> 
                TelephonyManager telephonyManager, FalsingCollector falsingCollector,

                EmergencyButtonController.Factory emergencyButtonControllerFactory,

                DevicePostureController devicePostureController,

                StatusBarKeyguardViewManager statusBarKeyguardViewManager) {

                KeyguardViewController keyguardViewController) {

            mKeyguardUpdateMonitor = keyguardUpdateMonitor;

            mLockPatternUtils = lockPatternUtils;

            mLatencyTracker = latencyTracker;
@@ -194,7 +193,7 @@ public abstract class KeyguardInputViewController<T extends KeyguardInputView> 
            mEmergencyButtonControllerFactory = emergencyButtonControllerFactory;

            mFalsingCollector = falsingCollector;

            mDevicePostureController = devicePostureController;

            mStatusBarKeyguardViewManager = statusBarKeyguardViewManager;

            mKeyguardViewController = keyguardViewController;

        }



        /** Create a new {@link KeyguardInputViewController}. */
@@ -215,7 +214,7 @@ public abstract class KeyguardInputViewController<T extends KeyguardInputView> 
                        mKeyguardUpdateMonitor, securityMode, mLockPatternUtils,

                        keyguardSecurityCallback, mMessageAreaControllerFactory, mLatencyTracker,

                        mInputMethodManager, emergencyButtonController, mMainExecutor, mResources,

                        mFalsingCollector, mStatusBarKeyguardViewManager);

                        mFalsingCollector, mKeyguardViewController);



            } else if (keyguardInputView instanceof KeyguardPINView) {

                return new KeyguardPinViewController((KeyguardPINView) keyguardInputView,

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

+4 −5
Original line number Diff line number Diff line
@@ -43,7 +43,6 @@ import com.android.settingslib.Utils; 
import com.android.systemui.R;

import com.android.systemui.classifier.FalsingCollector;

import com.android.systemui.dagger.qualifiers.Main;

import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager;

import com.android.systemui.util.concurrency.DelayableExecutor;



import java.util.List;
@@ -56,7 +55,7 @@ public class KeyguardPasswordViewController 
    private final KeyguardSecurityCallback mKeyguardSecurityCallback;

    private final InputMethodManager mInputMethodManager;

    private final DelayableExecutor mMainExecutor;

    private final StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;

    private final KeyguardViewController mKeyguardViewController;

    private final boolean mShowImeAtScreenOn;

    private EditText mPasswordEntry;

    private ImageView mSwitchImeButton;
@@ -119,14 +118,14 @@ public class KeyguardPasswordViewController 
            @Main DelayableExecutor mainExecutor,

            @Main Resources resources,

            FalsingCollector falsingCollector,

            StatusBarKeyguardViewManager statusBarKeyguardViewManager) {

            KeyguardViewController keyguardViewController) {

        super(view, keyguardUpdateMonitor, securityMode, lockPatternUtils, keyguardSecurityCallback,

                messageAreaControllerFactory, latencyTracker, falsingCollector,

                emergencyButtonController);

        mKeyguardSecurityCallback = keyguardSecurityCallback;

        mInputMethodManager = inputMethodManager;

        mMainExecutor = mainExecutor;

        mStatusBarKeyguardViewManager = statusBarKeyguardViewManager;

        mKeyguardViewController = keyguardViewController;

        mShowImeAtScreenOn = resources.getBoolean(R.bool.kg_show_ime_at_screen_on);

        mPasswordEntry = mView.findViewById(mView.getPasswordTextViewId());

        mSwitchImeButton = mView.findViewById(R.id.switch_ime_button);
@@ -209,7 +208,7 @@ public class KeyguardPasswordViewController 
    }



    private void showInput() {

        if (!mStatusBarKeyguardViewManager.isBouncerShowing()) {

        if (!mKeyguardViewController.isBouncerShowing()) {

            return;

        }

packages/SystemUI/tests/src/com/android/keyguard/KeyguardPasswordViewControllerTest.kt

+4 −5
Original line number Diff line number Diff line
@@ -25,7 +25,6 @@ import com.android.internal.widget.LockPatternUtils 
import com.android.systemui.R

import com.android.systemui.SysuiTestCase

import com.android.systemui.classifier.FalsingCollector

import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager

import com.android.systemui.util.concurrency.DelayableExecutor

import org.junit.Before

import org.junit.Test
@@ -63,7 +62,7 @@ class KeyguardPasswordViewControllerTest : SysuiTestCase() { 
    @Mock

    lateinit var falsingCollector: FalsingCollector

    @Mock

    lateinit var statusBarKeyguardViewManager: StatusBarKeyguardViewManager

    lateinit var keyguardViewController: KeyguardViewController

    @Mock

    private lateinit var mKeyguardMessageArea: KeyguardMessageArea

    @Mock
@@ -92,13 +91,13 @@ class KeyguardPasswordViewControllerTest : SysuiTestCase() { 
                mainExecutor,

                mContext.resources,

                falsingCollector,

                statusBarKeyguardViewManager

                keyguardViewController

        )

    }



    @Test

    fun testFocusWhenBouncerIsShown() {

        Mockito.`when`(statusBarKeyguardViewManager.isBouncerShowing).thenReturn(true)

        Mockito.`when`(keyguardViewController.isBouncerShowing).thenReturn(true)

        Mockito.`when`(keyguardPasswordView.isShown).thenReturn(true)

        keyguardPasswordViewController.onResume(KeyguardSecurityView.VIEW_REVEALED)

        keyguardPasswordView.post { verify(keyguardPasswordView).requestFocus() }
@@ -106,7 +105,7 @@ class KeyguardPasswordViewControllerTest : SysuiTestCase() { 


    @Test

    fun testDoNotFocusWhenBouncerIsHidden() {

        Mockito.`when`(statusBarKeyguardViewManager.isBouncerShowing).thenReturn(false)

        Mockito.`when`(keyguardViewController.isBouncerShowing).thenReturn(false)

        Mockito.`when`(keyguardPasswordView.isShown).thenReturn(true)

        keyguardPasswordViewController.onResume(KeyguardSecurityView.VIEW_REVEALED)

        verify(keyguardPasswordView, never()).requestFocus()

packages/SystemUI/tests/src/com/android/keyguard/KeyguardSecurityContainerControllerTest.java

+2 −3
Original line number Diff line number Diff line
@@ -51,7 +51,6 @@ import com.android.systemui.classifier.FalsingCollector; 
import com.android.systemui.flags.FeatureFlags;

import com.android.systemui.log.SessionTracker;

import com.android.systemui.plugins.FalsingManager;

import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager;

import com.android.systemui.statusbar.policy.ConfigurationController;

import com.android.systemui.statusbar.policy.KeyguardStateController;

import com.android.systemui.statusbar.policy.UserSwitcherController;
@@ -125,7 +124,7 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 
    @Mock

    private SessionTracker mSessionTracker;

    @Mock

    private StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;

    private KeyguardViewController mKeyguardViewController;

    private Configuration mConfiguration;



    private KeyguardSecurityContainerController mKeyguardSecurityContainerController;
@@ -153,7 +152,7 @@ public class KeyguardSecurityContainerControllerTest extends SysuiTestCase { 
                (KeyguardPasswordView) mKeyguardPasswordView, mKeyguardUpdateMonitor,

                SecurityMode.Password, mLockPatternUtils, null,

                mKeyguardMessageAreaControllerFactory, null, null, mEmergencyButtonController,

                null, mock(Resources.class), null, mStatusBarKeyguardViewManager);

                null, mock(Resources.class), null, mKeyguardViewController);



        mKeyguardSecurityContainerController = new KeyguardSecurityContainerController.Factory(

                mView, mAdminSecondaryLockScreenControllerFactory, mLockPatternUtils,