diff options
author | Karishma Vakil <kvakil@google.com> | 2023-01-30 21:39:18 +0000 |
---|---|---|
committer | Karishma Vakil <kvakil@google.com> | 2023-02-02 17:37:50 +0000 |
commit | 2be3f46bdfb49c79808188160d1e411f171885f7 (patch) | |
tree | 50041d12d059be83fbe4dc3d4feaa5e298bc23ae /PermissionController/src/com/android/permissioncontroller/permission/model | |
parent | fe7915137955546320455a79872f1daf993bf8f7 (diff) | |
download | Permission-2be3f46bdfb49c79808188160d1e411f171885f7.tar.gz |
[Safety Labels] Redirect to App location permission page instead of
general App permissions page from data sharing updates
- Also implement minor cleanup by using safety label lib constant
instead of hardcoded value for location category.
Bug: 263153040
Bug: 267177476
Test: atest CtsPermission3TestCases:AppDataSharingUpdatesTest
Change-Id: Ibe14c92a7e75b16aa10fd2b6f1d9b7a853db7e1d
Diffstat (limited to 'PermissionController/src/com/android/permissioncontroller/permission/model')
-rw-r--r-- | PermissionController/src/com/android/permissioncontroller/permission/model/v34/AppDataSharingUpdate.kt | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/PermissionController/src/com/android/permissioncontroller/permission/model/v34/AppDataSharingUpdate.kt b/PermissionController/src/com/android/permissioncontroller/permission/model/v34/AppDataSharingUpdate.kt index 9288ccc4c..ab0de7205 100644 --- a/PermissionController/src/com/android/permissioncontroller/permission/model/v34/AppDataSharingUpdate.kt +++ b/PermissionController/src/com/android/permissioncontroller/permission/model/v34/AppDataSharingUpdate.kt @@ -18,6 +18,7 @@ package com.android.permissioncontroller.permission.model.v34 import android.os.Build import androidx.annotation.RequiresApi +import com.android.permission.safetylabel.DataCategoryConstants.CATEGORY_LOCATION import com.android.permissioncontroller.permission.model.v34.DataSharingUpdateType.ADDS_ADVERTISING_PURPOSE import com.android.permissioncontroller.permission.model.v34.DataSharingUpdateType.ADDS_SHARING_WITHOUT_ADVERTISING_PURPOSE import com.android.permissioncontroller.permission.model.v34.DataSharingUpdateType.ADDS_SHARING_WITH_ADVERTISING_PURPOSE @@ -47,7 +48,7 @@ data class AppDataSharingUpdate( fun AppSafetyLabelDiff.buildUpdateIfSignificantChange(): AppDataSharingUpdate? { // In Android U, only updates for the location data category will be displayed in // the UI. - val updates = getUpdatesForCategories(listOf(LOCATION_CATEGORY)) + val updates = getUpdatesForCategories(listOf(CATEGORY_LOCATION)) return if (updates.isEmpty()) null else AppDataSharingUpdate(safetyLabelBefore.appInfo.packageName, updates) @@ -87,9 +88,6 @@ data class AppDataSharingUpdate( return categoryUpdateMap } - // TODO(b/263153040): Use categories from safety label library. - const val LOCATION_CATEGORY = "location" - private fun SafetyLabel.sharesData(category: String) = dataLabel.dataShared.containsKey(category) |