diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImpl.kt b/packages/SystemUI/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImpl.kt index 32c64f4575015..8c61ada3f8ef8 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImpl.kt +++ b/packages/SystemUI/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImpl.kt @@ -36,6 +36,7 @@ import com.android.systemui.dump.DumpManager import com.android.systemui.settings.UserTracker import com.android.systemui.util.settings.GlobalSettings import com.android.systemui.util.settings.SecureSettings +import com.android.systemui.util.wrapper.BuildInfo import java.io.PrintWriter import java.util.concurrent.Executor import java.util.concurrent.atomic.AtomicBoolean @@ -47,6 +48,7 @@ open class DeviceProvisionedControllerImpl @Inject constructor( private val globalSettings: GlobalSettings, private val userTracker: UserTracker, private val dumpManager: DumpManager, + private val buildInfo: BuildInfo, @Background private val backgroundHandler: Handler, @Main private val mainExecutor: Executor ) : DeviceProvisionedController, @@ -187,7 +189,7 @@ open class DeviceProvisionedControllerImpl @Inject constructor( } override fun isFrpActive(): Boolean { - return frpActive.get() + return frpActive.get() && !buildInfo.isDebuggable } override fun isUserSetup(user: Int): Boolean { diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImplTest.kt b/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImplTest.kt index 6980a0b4565e9..6094135c63641 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImplTest.kt +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/DeviceProvisionedControllerImplTest.kt @@ -27,8 +27,10 @@ import com.android.systemui.settings.UserTracker import com.android.systemui.util.concurrency.FakeExecutor import com.android.systemui.util.mockito.any import com.android.systemui.util.mockito.capture +import com.android.systemui.util.mockito.whenever import com.android.systemui.util.settings.FakeSettings import com.android.systemui.util.time.FakeSystemClock +import com.android.systemui.util.wrapper.BuildInfo import com.google.common.truth.Truth.assertThat import org.junit.Before import org.junit.Test @@ -36,9 +38,9 @@ import org.junit.runner.RunWith import org.mockito.ArgumentCaptor import org.mockito.Captor import org.mockito.Mock -import org.mockito.Mockito.`when` import org.mockito.Mockito.never import org.mockito.Mockito.verify +import org.mockito.Mockito.`when` import org.mockito.MockitoAnnotations @SmallTest @@ -58,6 +60,8 @@ class DeviceProvisionedControllerImplTest : SysuiTestCase() { private lateinit var dumpManager: DumpManager @Mock private lateinit var listener: DeviceProvisionedController.DeviceProvisionedListener + @Mock + private lateinit var buildInfo: BuildInfo @Captor private lateinit var userTrackerCallbackCaptor: ArgumentCaptor @@ -72,12 +76,13 @@ class DeviceProvisionedControllerImplTest : SysuiTestCase() { mainExecutor = FakeExecutor(FakeSystemClock()) settings = FakeSettings() `when`(userTracker.userId).thenReturn(START_USER) - + whenever(buildInfo.isDebuggable).thenReturn(false) controller = DeviceProvisionedControllerImpl( settings, settings, userTracker, dumpManager, + buildInfo, Handler(testableLooper.looper), mainExecutor )