summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Pechetty Sravani <pechetty@google.com> 2024-05-23 02:32:41 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2024-05-23 02:32:41 +0000
commit54e8c2dbbefd68e16b03fd84e1481edb0affdc7b (patch)
tree2db21d5e8d8a7d65fea04f050c955966e1709e06
parent0042d18c34b0b560c8f4cc9811890af4a31bd2ba (diff)
parenta51c813548262092668cf736d026f0586086b9aa (diff)
Merge "Revert "Use APIs for clipboard in PrivacyControlsViewModel"" into main
-rw-r--r--PermissionController/AndroidManifest.xml1
-rw-r--r--PermissionController/src/com/android/permissioncontroller/safetycenter/ui/model/PrivacyControlsViewModel.kt18
-rw-r--r--flags/flags.aconfig11
3 files changed, 0 insertions, 30 deletions
diff --git a/PermissionController/AndroidManifest.xml b/PermissionController/AndroidManifest.xml
index c0d6c2907..f1e84500e 100644
--- a/PermissionController/AndroidManifest.xml
+++ b/PermissionController/AndroidManifest.xml
@@ -68,7 +68,6 @@
<uses-permission android:name="android.permission.NFC_PREFERRED_PAYMENT_INFO" />
<uses-permission android:name="android.permission.MANAGE_ENHANCED_CONFIRMATION_STATES" />
<uses-permission android:name="android.permission.READ_SYSTEM_GRAMMATICAL_GENDER" />
- <uses-permission android:name="android.permission.MANAGE_CLIPBOARD_ACCESS_NOTIFICATION" />
<application android:name="com.android.permissioncontroller.PermissionControllerApplication"
android:label="@string/app_name"
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/model/PrivacyControlsViewModel.kt b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/model/PrivacyControlsViewModel.kt
index 41b16e8be..2f5702321 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/model/PrivacyControlsViewModel.kt
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/model/PrivacyControlsViewModel.kt
@@ -16,9 +16,7 @@
package com.android.permissioncontroller.safetycenter.ui.model
-import android.annotation.SuppressLint
import android.app.Application
-import android.content.ClipboardManager
import android.content.Intent
import android.hardware.SensorPrivacyManager
import android.hardware.SensorPrivacyManager.Sensors
@@ -34,8 +32,6 @@ import androidx.fragment.app.Fragment
import androidx.lifecycle.AndroidViewModel
import androidx.lifecycle.ViewModel
import androidx.lifecycle.ViewModelProvider
-import com.android.modules.utils.build.SdkLevel
-import com.android.permission.flags.Flags
import com.android.permissioncontroller.R
import com.android.permissioncontroller.permission.data.SmartUpdateMediatorLiveData
import com.android.settingslib.RestrictedLockUtils
@@ -43,15 +39,10 @@ import com.android.settingslib.RestrictedLockUtils.EnforcedAdmin
/** Viewmodel for the privacy controls page. */
@RequiresApi(Build.VERSION_CODES.TIRAMISU)
-// Suppress warnings related to the camera/mic privacy and clipboard privacy APIs. The PC has the
-// permissions.
-@SuppressLint("MissingPermission")
class PrivacyControlsViewModel(private val app: Application) : AndroidViewModel(app) {
private val sensorPrivacyManager: SensorPrivacyManager =
app.getSystemService(SensorPrivacyManager::class.java)!!
- private val clipboardManager: ClipboardManager =
- app.getSystemService(ClipboardManager::class.java)!!
private val userManager: UserManager = app.getSystemService(UserManager::class.java)!!
private val CONFIG_CLIPBOARD_SHOW_ACCESS_NOTIFICATIONS =
@@ -170,10 +161,6 @@ class PrivacyControlsViewModel(private val app: Application) : AndroidViewModel(
}
private fun isClipboardEnabled(): Boolean {
- if (SdkLevel.isAtLeastU() && Flags.useApiForClipboardPrivacyToggle()) {
- return clipboardManager.areClipboardAccessNotificationsEnabled()
- }
-
val clipboardDefaultEnabled =
DeviceConfig.getBoolean(
DeviceConfig.NAMESPACE_CLIPBOARD,
@@ -189,11 +176,6 @@ class PrivacyControlsViewModel(private val app: Application) : AndroidViewModel(
}
private fun toggleClipboard() {
- if (SdkLevel.isAtLeastU() && Flags.useApiForClipboardPrivacyToggle()) {
- clipboardManager.setClipboardAccessNotificationsEnabled(!isClipboardEnabled())
- return
- }
-
val newState = if (isClipboardEnabled()) 0 else 1
Settings.Secure.putInt(
app.contentResolver,
diff --git a/flags/flags.aconfig b/flags/flags.aconfig
index a063e4103..a81de8144 100644
--- a/flags/flags.aconfig
+++ b/flags/flags.aconfig
@@ -29,17 +29,6 @@ flag {
}
flag {
- name: "use_api_for_clipboard_privacy_toggle"
- namespace: "permissions"
- description: "This flag guards whether the Privacy Controls page uses a hardcoded string or an API to control clipboard privacy"
- bug: "263305143"
- metadata {
- purpose: PURPOSE_BUGFIX
- }
- is_fixed_read_only: true
-}
-
-flag {
name: "archiving_read_only"
is_exported: true
namespace: "permissions"