From cf0e0b78e030abdf487577571eb9b37aee3bfe48 Mon Sep 17 00:00:00 2001
From: Yi-an Chen <theianchen@google.com>
Date: Tue, 11 Mar 2025 21:03:14 +0000
Subject: Replace allowPrivateProfile flag with sdk version check

b/398830041 currently impacts flag value matching for mainline module on
V platform. According to trunk stable team, these flag checks can be
replaced with sdk version check. Hence removing the flag checks in this
CL

Bug: 396234389
Test: Build
Flag: EXEMPT bugfix
Relnote: N/A
Change-Id: Id351f1516d9db763e8d765da552d4da2182300b9
---
 .../com/android/permissioncontroller/role/utils/UserUtils.java    | 2 +-
 .../safetycenter/service/SafetyCenterSearchIndexablesProvider.kt  | 4 +---
 .../safetycenter/ui/SafetySubpageEntryPreference.kt               | 8 +++-----
 .../safetycenter/ui/StaticSafetyEntryPreference.java              | 3 +--
 4 files changed, 6 insertions(+), 11 deletions(-)

(limited to 'PermissionController/src')

diff --git a/PermissionController/src/com/android/permissioncontroller/role/utils/UserUtils.java b/PermissionController/src/com/android/permissioncontroller/role/utils/UserUtils.java
index 4d778befb..10e8dd0e1 100644
--- a/PermissionController/src/com/android/permissioncontroller/role/utils/UserUtils.java
+++ b/PermissionController/src/com/android/permissioncontroller/role/utils/UserUtils.java
@@ -139,7 +139,7 @@ public class UserUtils {
      */
     private static boolean isPrivateProfile(@NonNull UserHandle userHandle,
             @NonNull Context context) {
-        if (!SdkLevel.isAtLeastV() || !android.os.Flags.allowPrivateProfile()) {
+        if (!SdkLevel.isAtLeastV()) {
             return false;
         }
         Context userContext = context.createContextAsUser(userHandle, /* flags= */ 0);
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/service/SafetyCenterSearchIndexablesProvider.kt b/PermissionController/src/com/android/permissioncontroller/safetycenter/service/SafetyCenterSearchIndexablesProvider.kt
index 8d7d96f99..307ef9cf6 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/service/SafetyCenterSearchIndexablesProvider.kt
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/service/SafetyCenterSearchIndexablesProvider.kt
@@ -331,9 +331,7 @@ class SafetyCenterSearchIndexablesProvider : BaseSearchIndexablesProvider() {
         get() = SafetyCenterIds.entryIdFromString(id)
 
     private fun isPrivateProfileSupported(): Boolean {
-        return SdkLevel.isAtLeastV() &&
-            com.android.permission.flags.Flags.privateProfileSupported() &&
-            android.os.Flags.allowPrivateProfile()
+        return SdkLevel.isAtLeastV() && com.android.permission.flags.Flags.privateProfileSupported()
     }
 
     companion object {
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetySubpageEntryPreference.kt b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetySubpageEntryPreference.kt
index b89abde13..551f676f1 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetySubpageEntryPreference.kt
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetySubpageEntryPreference.kt
@@ -49,7 +49,7 @@ class SafetySubpageEntryPreference(
     context: Context,
     private val launchTaskId: Int?,
     private val entry: SafetyCenterEntry,
-    private val viewModel: SafetyCenterViewModel
+    private val viewModel: SafetyCenterViewModel,
 ) : TwoTargetPreference(context), ComparablePreference {
 
     init {
@@ -108,9 +108,7 @@ class SafetySubpageEntryPreference(
     }
 
     private fun isPrivateProfileSupported(): Boolean {
-        return SdkLevel.isAtLeastV() &&
-            com.android.permission.flags.Flags.privateProfileSupported() &&
-            android.os.Flags.allowPrivateProfile()
+        return SdkLevel.isAtLeastV() && com.android.permission.flags.Flags.privateProfileSupported()
     }
 
     override fun onBindViewHolder(holder: PreferenceViewHolder) {
@@ -125,7 +123,7 @@ class SafetySubpageEntryPreference(
                     PendingIntentSender.send(iconAction.pendingIntent, launchTaskId)
                     viewModel.interactionLogger.recordForEntry(
                         Action.ENTRY_ICON_ACTION_CLICKED,
-                        entry
+                        entry,
                     )
                 } catch (ex: Exception) {
                     Log.e(TAG, "Failed to execute icon action intent for $entry", ex)
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/StaticSafetyEntryPreference.java b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/StaticSafetyEntryPreference.java
index 87d8744a8..0da8286f6 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/StaticSafetyEntryPreference.java
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/StaticSafetyEntryPreference.java
@@ -98,8 +98,7 @@ public class StaticSafetyEntryPreference extends Preference implements Comparabl
 
     private Boolean isPrivateProfileSupported() {
         return SdkLevel.isAtLeastV()
-                && com.android.permission.flags.Flags.privateProfileSupported()
-                && android.os.Flags.allowPrivateProfile();
+                && com.android.permission.flags.Flags.privateProfileSupported();
     }
 
     @Override
-- 
cgit v1.2.3-59-g8ed1b