Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a3f652bf authored by Peter Wang's avatar Peter Wang
Browse files

Revert "[Telephony Mainline] Rename getActiveAndHiddenSubscriptionIdList and" 

This reverts commit 9487c9e5.

Reason for revert: ag/10731793 introduced a new system-current.txt that breaks build

Change-Id: I07f5a45ab454230b82aa336d4880fd2dcb823bda
parent 9487c9e5

api/system-current.txt

+2 −2
Original line number Diff line number Diff line
@@ -11311,10 +11311,10 @@ package android.telephony { 
  public class SubscriptionManager {

    method @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public boolean canDisablePhysicalSubscription();

    method public boolean canManageSubscription(@NonNull android.telephony.SubscriptionInfo, @NonNull String);

    method @NonNull @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public int[] getActiveSubscriptionIdList();

    method @NonNull public int[] getActiveAndHiddenSubscriptionIdList();

    method @NonNull public int[] getActiveSubscriptionIdList();

    method @Nullable @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public android.telephony.SubscriptionInfo getActiveSubscriptionInfoForIcc(@NonNull String);

    method public java.util.List<android.telephony.SubscriptionInfo> getAvailableSubscriptionInfoList();

    method @NonNull @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public int[] getCompleteActiveSubscriptionIdList();

    method @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public int getEnabledSubscriptionId(int);

    method @NonNull public static android.content.res.Resources getResourcesForSubId(@NonNull android.content.Context, int);

    method @RequiresPermission(android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE) public boolean isSubscriptionEnabled(int);

services/core/java/com/android/server/TelephonyRegistry.java

+1 −1
Original line number Diff line number Diff line
@@ -1394,7 +1394,7 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { 
    public void notifyCarrierNetworkChange(boolean active) {

        // only CarrierService with carrier privilege rule should have the permission

        int[] subIds = Arrays.stream(SubscriptionManager.from(mContext)

                    .getCompleteActiveSubscriptionIdList())

                    .getActiveAndHiddenSubscriptionIdList())

                    .filter(i -> TelephonyPermissions.checkCarrierPrivilegeForSubId(mContext,

                            i)).toArray();

        if (ArrayUtils.isEmpty(subIds)) {

telephony/common/com/android/internal/telephony/TelephonyPermissions.java

+1 −1
Original line number Diff line number Diff line
@@ -611,7 +611,7 @@ public final class TelephonyPermissions { 
    private static boolean checkCarrierPrivilegeForAnySubId(Context context, int uid) {

        SubscriptionManager sm = (SubscriptionManager) context.getSystemService(

                Context.TELEPHONY_SUBSCRIPTION_SERVICE);

        int[] activeSubIds = sm.getCompleteActiveSubscriptionIdList();

        int[] activeSubIds = sm.getActiveAndHiddenSubscriptionIdList();

        for (int activeSubId : activeSubIds) {

            if (getCarrierPrivilegeStatus(context, activeSubId, uid)

                    == TelephonyManager.CARRIER_PRIVILEGE_STATUS_HAS_ACCESS) {

telephony/java/android/telephony/SubscriptionManager.java

+1 −3
Original line number Diff line number Diff line
@@ -2161,7 +2161,6 @@ public class SubscriptionManager { 
     * @hide

     */

    @SystemApi

    @RequiresPermission(Manifest.permission.READ_PRIVILEGED_PHONE_STATE)

    public @NonNull int[] getActiveSubscriptionIdList() {

        return getActiveSubscriptionIdList(/* visibleOnly */ true);

    }
@@ -2179,8 +2178,7 @@ public class SubscriptionManager { 
     * @hide

     */

    @SystemApi

    @RequiresPermission(Manifest.permission.READ_PRIVILEGED_PHONE_STATE)

    public @NonNull int[] getCompleteActiveSubscriptionIdList() {

    public @NonNull int[] getActiveAndHiddenSubscriptionIdList() {

        return getActiveSubscriptionIdList(/* visibleOnly */false);

    }