diff --git a/src/com/android/settings/spa/app/specialaccess/InstallUnknownApps.kt b/src/com/android/settings/spa/app/specialaccess/InstallUnknownApps.kt index dc983307197..3e9058f25ff 100644 --- a/src/com/android/settings/spa/app/specialaccess/InstallUnknownApps.kt +++ b/src/com/android/settings/spa/app/specialaccess/InstallUnknownApps.kt @@ -26,7 +26,7 @@ import android.content.pm.ApplicationInfo import android.os.UserManager import androidx.compose.runtime.Composable import com.android.settings.R -import com.android.settingslib.spa.livedata.observeAsCallback +import com.android.settingslib.spa.lifecycle.collectAsCallbackWithLifecycle import com.android.settingslib.spaprivileged.model.app.AppOpsController import com.android.settingslib.spaprivileged.model.app.AppRecord import com.android.settingslib.spaprivileged.model.app.userId @@ -81,7 +81,7 @@ class InstallUnknownAppsListModel(private val context: Context) : @Composable override fun isAllowed(record: InstallUnknownAppsRecord) = - record.appOpsController.isAllowed.observeAsCallback() + record.appOpsController.isAllowed.collectAsCallbackWithLifecycle() override fun isChangeable(record: InstallUnknownAppsRecord) = isChangeable(record, getPotentialPackageNames(record.app.userId)) diff --git a/src/com/android/settings/spa/app/specialaccess/PictureInPicture.kt b/src/com/android/settings/spa/app/specialaccess/PictureInPicture.kt index fe8f1031268..7885b869b83 100644 --- a/src/com/android/settings/spa/app/specialaccess/PictureInPicture.kt +++ b/src/com/android/settings/spa/app/specialaccess/PictureInPicture.kt @@ -27,7 +27,7 @@ import android.content.pm.PackageManager.PackageInfoFlags import android.util.Log import androidx.compose.runtime.Composable import com.android.settings.R -import com.android.settingslib.spa.livedata.observeAsCallback +import com.android.settingslib.spa.lifecycle.collectAsCallbackWithLifecycle import com.android.settingslib.spaprivileged.model.app.AppOpsController import com.android.settingslib.spaprivileged.model.app.AppRecord import com.android.settingslib.spaprivileged.model.app.installed @@ -92,7 +92,7 @@ class PictureInPictureListModel(private val context: Context) : @Composable override fun isAllowed(record: PictureInPictureRecord) = - record.appOpsController.isAllowed.observeAsCallback() + record.appOpsController.isAllowed.collectAsCallbackWithLifecycle() override fun isChangeable(record: PictureInPictureRecord) = record.isSupport diff --git a/tests/spa_unit/src/com/android/settings/spa/app/WifiControlAppListModelTest.kt b/tests/spa_unit/src/com/android/settings/spa/app/WifiControlAppListModelTest.kt index 74aa86175eb..537764afe7c 100644 --- a/tests/spa_unit/src/com/android/settings/spa/app/WifiControlAppListModelTest.kt +++ b/tests/spa_unit/src/com/android/settings/spa/app/WifiControlAppListModelTest.kt @@ -21,7 +21,6 @@ import android.app.AppOpsManager import android.content.Context import android.content.pm.ApplicationInfo import androidx.compose.ui.test.junit4.createComposeRule -import androidx.lifecycle.MutableLiveData import androidx.test.core.app.ApplicationProvider import androidx.test.ext.junit.runners.AndroidJUnit4 import com.android.settingslib.spa.testutils.firstWithTimeoutOrNull @@ -270,7 +269,7 @@ class WifiControlAppListModelTest { private class FakeAppOpsController(private val fakeMode: Int) : IAppOpsController { var setAllowedCalledWith: Boolean? = null - override val mode = MutableLiveData(fakeMode) + override val mode = flowOf(fakeMode) override fun setAllowed(allowed: Boolean) { setAllowedCalledWith = allowed diff --git a/tests/spa_unit/src/com/android/settings/spa/app/specialaccess/MediaRoutingControlTest.kt b/tests/spa_unit/src/com/android/settings/spa/app/specialaccess/MediaRoutingControlTest.kt index 2f4740e65d4..990ec5ce417 100644 --- a/tests/spa_unit/src/com/android/settings/spa/app/specialaccess/MediaRoutingControlTest.kt +++ b/tests/spa_unit/src/com/android/settings/spa/app/specialaccess/MediaRoutingControlTest.kt @@ -24,7 +24,6 @@ import android.companion.AssociationRequest import android.content.Context import android.content.pm.ApplicationInfo import android.platform.test.flag.junit.SetFlagsRule -import androidx.lifecycle.MutableLiveData import androidx.test.core.app.ApplicationProvider import androidx.test.ext.junit.runners.AndroidJUnit4 import com.android.media.flags.Flags @@ -33,6 +32,7 @@ import com.android.settings.testutils.FakeFeatureFactory import com.android.settingslib.spaprivileged.model.app.IAppOpsController import com.android.settingslib.spaprivileged.template.app.AppOpPermissionRecord import com.google.common.truth.Truth.assertThat +import kotlinx.coroutines.flow.MutableStateFlow import org.junit.Before import org.junit.Rule import org.junit.Test @@ -223,16 +223,13 @@ class MediaRoutingControlTest { private class FakeAppOpsController(fakeMode: Int) : IAppOpsController { - override val mode = MutableLiveData(fakeMode) + override val mode = MutableStateFlow(fakeMode) override fun setAllowed(allowed: Boolean) { - if (allowed) - mode.postValue(AppOpsManager.MODE_ALLOWED) - else - mode.postValue(AppOpsManager.MODE_ERRORED) + mode.value = if (allowed) AppOpsManager.MODE_ALLOWED else AppOpsManager.MODE_ERRORED } - override fun getMode(): Int = mode.value!! + override fun getMode(): Int = mode.value } companion object {