PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove EXTRA_USER_HANDLE from startSpaActivityForApp

Browse Source 
Bug: 263553430
Test: Unit test
Change-Id: I6739d4518eac00cfe22905c1a6589b6e9f91a1d3
This commit is contained in:
Chaohui Wang
2023-01-04 16:26:04 +08:00
parent e4f2fb56f9
commit afeb7496bb
2 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/com/android/settings/spa/SpaActivity.kt
View File

@@ -34,10 +34,7 @@ class SpaActivity : BrowseActivity() {
@JvmStatic
fun Context.startSpaActivityForApp(destinationPrefix: String, intent: Intent): Boolean {
val packageName = intent.data?.schemeSpecificPart ?: return false
val userId = intent.getParcelableExtra(Intent.EXTRA_USER_HANDLE, UserHandle::class.java)
?.identifier
?: UserHandle.myUserId()
startSpaActivity("$destinationPrefix/$packageName/$userId")
startSpaActivity("$destinationPrefix/$packageName/${UserHandle.myUserId()}")
return true
}
}

4
tests/spa_unit/src/com/android/settings/spa/SpaActivityTest.kt
View File

@@ -57,7 +57,6 @@ class SpaActivityTest {
fun startSpaActivityForApp() {
val intent = Intent().apply {
data = Uri.parse("package:$PACKAGE_NAME")
putExtra(Intent.EXTRA_USER_HANDLE, UserHandle.of(USER_ID))
}
context.startSpaActivityForApp(DESTINATION, intent)
@@ -67,12 +66,11 @@ class SpaActivityTest {
val capturedIntent = intentCaptor.value
assertThat(capturedIntent.component?.className).isEqualTo(SpaActivity::class.qualifiedName)
assertThat(capturedIntent.getStringExtra(KEY_DESTINATION))
.isEqualTo("Destination/package.name/1")
.isEqualTo("Destination/package.name/${UserHandle.myUserId()}")
}
private companion object {
const val DESTINATION = "Destination"
const val PACKAGE_NAME = "package.name"
const val USER_ID = 1
}
}