commit 343cab409a3f78adc9739be8867abf3664681bb3
author Ling Ma <linggm@google.com> Mon Oct 30 22:36:44 2023 +0000
committer Ling Ma <linggm@google.com> Mon Oct 30 23:25:33 2023 +0000
tree 67fd78f400b24fdc80a9f0b04d07761a93c347b4
parent 5c2d62232ca008de55ce2c8735eb9701ad12b8a9

Revert "Limited profiles to access only associated subscriptions"

Revert submission 24907577-split_subscriptionManager

Reason for revert: 308469709#comment8

Reverted changes: /q/submissionid:24907577-split_subscriptionManager

Change-Id: Ib4cfd1c4df6f0f34756e3c1d6453ae3333ea515a

telephony/java/android/telephony/SubscriptionManager.java

diff --git a/telephony/java/android/telephony/SubscriptionManager.java b/telephony/java/android/telephony/SubscriptionManager.java
index f8608b8..8e90fe7 100644
--- a/telephony/java/android/telephony/SubscriptionManager.java
+++ b/telephony/java/android/telephony/SubscriptionManager.java
@@ -265,8 +265,8 @@
         }
     }
 
-    private static IntegerPropertyInvalidatedCache<Integer> sGetDefaultSubIdCacheAsUser =
-            new IntegerPropertyInvalidatedCache<>(ISub::getDefaultSubIdAsUser,
+    private static VoidPropertyInvalidatedCache<Integer> sGetDefaultSubIdCache =
+            new VoidPropertyInvalidatedCache<>(ISub::getDefaultSubId,
                     CACHE_KEY_SUBSCRIPTION_MANAGER_SERVICE_PROPERTY,
                     INVALID_SUBSCRIPTION_ID);
 
@@ -275,8 +275,8 @@
                     CACHE_KEY_SUBSCRIPTION_MANAGER_SERVICE_PROPERTY,
                     INVALID_SUBSCRIPTION_ID);
 
-    private static IntegerPropertyInvalidatedCache<Integer> sGetDefaultSmsSubIdCacheAsUser =
-            new IntegerPropertyInvalidatedCache<>(ISub::getDefaultSmsSubIdAsUser,
+    private static VoidPropertyInvalidatedCache<Integer> sGetDefaultSmsSubIdCache =
+            new VoidPropertyInvalidatedCache<>(ISub::getDefaultSmsSubId,
                     CACHE_KEY_SUBSCRIPTION_MANAGER_SERVICE_PROPERTY,
                     INVALID_SUBSCRIPTION_ID);
 
@@ -2309,7 +2309,7 @@
      * @return the "system" default subscription id.
      */
     public static int getDefaultSubscriptionId() {
-        return sGetDefaultSubIdCacheAsUser.query(Process.myUserHandle().getIdentifier());
+        return sGetDefaultSubIdCache.query(null);
     }
 
     /**
@@ -2325,7 +2325,7 @@
         try {
             ISub iSub = TelephonyManager.getSubscriptionService();
             if (iSub != null) {
-                subId = iSub.getDefaultVoiceSubIdAsUser(Process.myUserHandle().getIdentifier());
+                subId = iSub.getDefaultVoiceSubId();
             }
         } catch (RemoteException ex) {
             // ignore it
@@ -2397,7 +2397,7 @@
      * @return the default SMS subscription Id.
      */
     public static int getDefaultSmsSubscriptionId() {
-        return sGetDefaultSmsSubIdCacheAsUser.query(Process.myUserHandle().getIdentifier());
+        return sGetDefaultSmsSubIdCache.query(null);
     }
 
     /**
@@ -3927,10 +3927,10 @@
      * @hide
      */
     public static void disableCaching() {
-        sGetDefaultSubIdCacheAsUser.disableLocal();
+        sGetDefaultSubIdCache.disableLocal();
         sGetDefaultDataSubIdCache.disableLocal();
         sGetActiveDataSubscriptionIdCache.disableLocal();
-        sGetDefaultSmsSubIdCacheAsUser.disableLocal();
+        sGetDefaultSmsSubIdCache.disableLocal();
         sGetSlotIndexCache.disableLocal();
         sGetSubIdCache.disableLocal();
         sGetPhoneIdCache.disableLocal();
@@ -3941,10 +3941,10 @@
      *
      * @hide */
     public static void clearCaches() {
-        sGetDefaultSubIdCacheAsUser.clear();
+        sGetDefaultSubIdCache.clear();
         sGetDefaultDataSubIdCache.clear();
         sGetActiveDataSubscriptionIdCache.clear();
-        sGetDefaultSmsSubIdCacheAsUser.clear();
+        sGetDefaultSmsSubIdCache.clear();
         sGetSlotIndexCache.clear();
         sGetSubIdCache.clear();
         sGetPhoneIdCache.clear();

telephony/java/com/android/internal/telephony/ISub.aidl

diff --git a/telephony/java/com/android/internal/telephony/ISub.aidl b/telephony/java/com/android/internal/telephony/ISub.aidl
index d2dbeb7..a5a23e8 100644
--- a/telephony/java/com/android/internal/telephony/ISub.aidl
+++ b/telephony/java/com/android/internal/telephony/ISub.aidl
@@ -239,7 +239,6 @@
     int getSubId(int slotIndex);
 
     int getDefaultSubId();
-    int getDefaultSubIdAsUser(int userId);
 
     int getPhoneId(int subId);
 
@@ -252,12 +251,10 @@
     void setDefaultDataSubId(int subId);
 
     int getDefaultVoiceSubId();
-    int getDefaultVoiceSubIdAsUser(int userId);
 
     void setDefaultVoiceSubId(int subId);
 
     int getDefaultSmsSubId();
-    int getDefaultSmsSubIdAsUser(int userId);
 
     void setDefaultSmsSubId(int subId);