Change 'idx' to 'index' for SubscriptionManager api.
Sanket Padawe [Tue, 21 Mar 2017 17:57:55 +0000 (10:57 -0700)]
Bug: 35767068
Test: No build failure on update.
Change-Id: I56977b047512100d5fbe7eb8eb2df8aa1f773b8b

src/java/com/android/internal/telephony/SubscriptionController.java
src/java/com/android/internal/telephony/SubscriptionInfoUpdater.java
src/java/com/android/internal/telephony/cat/CatService.java
src/java/com/android/internal/telephony/ims/ImsResolver.java
src/java/com/android/internal/telephony/uicc/IccCardProxy.java
tests/telephonytests/src/com/android/internal/telephony/SubscriptionControllerTest.java
tests/telephonytests/src/com/android/internal/telephony/SubscriptionInfoUpdaterTest.java
tests/telephonytests/src/com/android/internal/telephony/ims/ImsResolverTest.java
tests/telephonytests/src/com/android/internal/telephony/mocks/SubscriptionControllerMock.java

index 2bb0fc3..5b85bf6 100644 (file)
@@ -58,7 +58,7 @@ import java.util.concurrent.ConcurrentHashMap;
  * SubscriptionController to provide an inter-process communication to
  * access Sms in Icc.
  *
- * Any setters which take subId, slotId or phoneId as a parameter will throw an exception if the
+ * Any setters which take subId, slotIndex or phoneId as a parameter will throw an exception if the
  * parameter equals the corresponding INVALID_XXX_ID or DEFAULT_XXX_ID.
  *
  * All getters will lookup the corresponding default if the parameter is DEFAULT_XXX_ID. Ie calling
@@ -66,9 +66,9 @@ import java.util.concurrent.ConcurrentHashMap;
  *
  * Finally, any getters which perform the mapping between subscriptions, slots and phones will
  * return the corresponding INVALID_XXX_ID if the parameter is INVALID_XXX_ID. All other getters
- * will fail and return the appropriate error value. Ie calling getSlotId(INVALID_SUBSCRIPTION_ID)
- * will return INVALID_SLOT_ID and calling getSubInfoForSubscriber(INVALID_SUBSCRIPTION_ID)
- * will return null.
+ * will fail and return the appropriate error value. Ie calling
+ * getSlotIndex(INVALID_SUBSCRIPTION_ID) will return INVALID_SIM_SLOT_INDEX and calling
+ * getSubInfoForSubscriber(INVALID_SUBSCRIPTION_ID) will return null.
  *
  */
 public class SubscriptionController extends ISub.Stub {
@@ -128,7 +128,7 @@ public class SubscriptionController extends ISub.Stub {
     private AppOpsManager mAppOps;
 
     // FIXME: Does not allow for multiple subs in a slot and change to SparseArray
-    private static Map<Integer, Integer> sSlotIdxToSubId =
+    private static Map<Integer, Integer> sSlotIndexToSubId =
             new ConcurrentHashMap<Integer, Integer>();
     private static int mDefaultFallbackSubId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;
     private static int mDefaultPhoneId = SubscriptionManager.DEFAULT_PHONE_INDEX;
@@ -185,7 +185,7 @@ public class SubscriptionController extends ISub.Stub {
     }
 
     private boolean isSubInfoReady() {
-        return sSlotIdxToSubId.size() > 0;
+        return sSlotIndexToSubId.size() > 0;
     }
 
     private SubscriptionController(Phone phone) {
@@ -466,13 +466,13 @@ public class SubscriptionController extends ISub.Stub {
     }
 
     /**
-     * Get the active SubscriptionInfo associated with the slotIdx
-     * @param slotIdx the slot which the subscription is inserted
+     * Get the active SubscriptionInfo associated with the slotIndex
+     * @param slotIndex the slot which the subscription is inserted
      * @param callingPackage The package making the IPC.
      * @return SubscriptionInfo, maybe null if its not active
      */
     @Override
-    public SubscriptionInfo getActiveSubscriptionInfoForSimSlotIndex(int slotIdx,
+    public SubscriptionInfo getActiveSubscriptionInfoForSimSlotIndex(int slotIndex,
             String callingPackage) {
         if (!canReadPhoneState(callingPackage, "getActiveSubscriptionInfoForSimSlotIndex")) {
             return null;
@@ -485,16 +485,16 @@ public class SubscriptionController extends ISub.Stub {
                     mContext.getOpPackageName());
             if (subList != null) {
                 for (SubscriptionInfo si : subList) {
-                    if (si.getSimSlotIndex() == slotIdx) {
+                    if (si.getSimSlotIndex() == slotIndex) {
                         if (DBG) {
-                            logd("[getActiveSubscriptionInfoForSimSlotIndex]+ slotIdx=" + slotIdx
-                                    + " subId=" + si);
+                            logd("[getActiveSubscriptionInfoForSimSlotIndex]+ slotIndex="
+                                    + slotIndex + " subId=" + si);
                         }
                         return si;
                     }
                 }
                 if (DBG) {
-                    logd("[getActiveSubscriptionInfoForSimSlotIndex]+ slotIdx=" + slotIdx
+                    logd("[getActiveSubscriptionInfoForSimSlotIndex]+ slotIndex=" + slotIndex
                             + " subId=null");
                 }
             } else {
@@ -666,13 +666,13 @@ public class SubscriptionController extends ISub.Stub {
     /**
      * Add a new SubInfoRecord to subinfo database if needed
      * @param iccId the IccId of the SIM card
-     * @param slotId the slot which the SIM is inserted
+     * @param slotIndex the slot which the SIM is inserted
      * @return 0 if success, < 0 on error.
      */
     @Override
-    public int addSubInfoRecord(String iccId, int slotId) {
+    public int addSubInfoRecord(String iccId, int slotIndex) {
         if (DBG) logdl("[addSubInfoRecord]+ iccId:" + SubscriptionInfo.givePrintableIccid(iccId) +
-                " slotId:" + slotId);
+                " slotIndex:" + slotIndex);
 
         enforceModifyPhoneState("addSubInfoRecord");
 
@@ -699,7 +699,7 @@ public class SubscriptionController extends ISub.Stub {
                     value.put(SubscriptionManager.ICC_ID, iccId);
                     // default SIM color differs between slots
                     value.put(SubscriptionManager.COLOR, color);
-                    value.put(SubscriptionManager.SIM_SLOT_INDEX, slotId);
+                    value.put(SubscriptionManager.SIM_SLOT_INDEX, slotIndex);
                     value.put(SubscriptionManager.CARRIER_NAME, "");
                     Uri uri = resolver.insert(SubscriptionManager.CONTENT_URI, value);
                     if (DBG) logdl("[addSubInfoRecord] New record created: " + uri);
@@ -709,8 +709,8 @@ public class SubscriptionController extends ISub.Stub {
                     int nameSource = cursor.getInt(2);
                     ContentValues value = new ContentValues();
 
-                    if (slotId != oldSimInfoId) {
-                        value.put(SubscriptionManager.SIM_SLOT_INDEX, slotId);
+                    if (slotIndex != oldSimInfoId) {
+                        value.put(SubscriptionManager.SIM_SLOT_INDEX, slotIndex);
                     }
 
                     if (nameSource != SubscriptionManager.NAME_SOURCE_USER_INPUT) {
@@ -733,30 +733,30 @@ public class SubscriptionController extends ISub.Stub {
 
             cursor = resolver.query(SubscriptionManager.CONTENT_URI, null,
                     SubscriptionManager.SIM_SLOT_INDEX + "=?",
-                    new String[] {String.valueOf(slotId)}, null);
+                    new String[] {String.valueOf(slotIndex)}, null);
             try {
                 if (cursor != null && cursor.moveToFirst()) {
                     do {
                         int subId = cursor.getInt(cursor.getColumnIndexOrThrow(
                                 SubscriptionManager.UNIQUE_KEY_SUBSCRIPTION_ID));
-                        // If sSlotIdxToSubId already has a valid subId for a slotId/phoneId,
-                        // do not add another subId for same slotId/phoneId.
-                        Integer currentSubId = sSlotIdxToSubId.get(slotId);
+                        // If sSlotIndexToSubId already has a valid subId for a slotIndex/phoneId,
+                        // do not add another subId for same slotIndex/phoneId.
+                        Integer currentSubId = sSlotIndexToSubId.get(slotIndex);
                         if (currentSubId == null
                                 || !SubscriptionManager.isValidSubscriptionId(currentSubId)) {
                             // TODO While two subs active, if user deactivats first
                             // one, need to update the default subId with second one.
 
-                            // FIXME: Currently we assume phoneId == slotId which in the future
+                            // FIXME: Currently we assume phoneId == slotIndex which in the future
                             // may not be true, for instance with multiple subs per slot.
                             // But is true at the moment.
-                            sSlotIdxToSubId.put(slotId, subId);
+                            sSlotIndexToSubId.put(slotIndex, subId);
                             int subIdCountMax = getActiveSubInfoCountMax();
                             int defaultSubId = getDefaultSubId();
                             if (DBG) {
                                 logdl("[addSubInfoRecord]"
-                                        + " sSlotIdxToSubId.size=" + sSlotIdxToSubId.size()
-                                        + " slotId=" + slotId + " subId=" + subId
+                                        + " sSlotIndexToSubId.size=" + sSlotIndexToSubId.size()
+                                        + " slotIndex=" + slotIndex + " subId=" + subId
                                         + " defaultSubId=" + defaultSubId + " simCount=" + subIdCountMax);
                             }
 
@@ -780,7 +780,7 @@ public class SubscriptionController extends ISub.Stub {
                                         + " && currentSubId is valid, IGNORE");
                             }
                         }
-                        if (DBG) logdl("[addSubInfoRecord] hashmap(" + slotId + "," + subId + ")");
+                        if (DBG) logdl("[addSubInfoRecord] hashmap(" + slotIndex + "," + subId + ")");
                     } while (cursor.moveToNext());
                 }
             } finally {
@@ -790,7 +790,7 @@ public class SubscriptionController extends ISub.Stub {
             }
 
             // Set Display name after sub id is set above so as to get valid simCarrierName
-            int subId = getSubIdUsingPhoneId(slotId);
+            int subId = getSubIdUsingPhoneId(slotIndex);
             if (!SubscriptionManager.isValidSubscriptionId(subId)) {
                 if (DBG) {
                     logdl("[addSubInfoRecord]- getSubId failed invalid subId = " + subId);
@@ -804,7 +804,7 @@ public class SubscriptionController extends ISub.Stub {
                 if (!TextUtils.isEmpty(simCarrierName)) {
                     nameToSet = simCarrierName;
                 } else {
-                    nameToSet = "CARD " + Integer.toString(slotId + 1);
+                    nameToSet = "CARD " + Integer.toString(slotIndex + 1);
                 }
 
                 ContentValues value = new ContentValues();
@@ -817,9 +817,9 @@ public class SubscriptionController extends ISub.Stub {
             }
 
             // Once the records are loaded, notify DcTracker
-            sPhones[slotId].updateDataConnectionTracker();
+            sPhones[slotIndex].updateDataConnectionTracker();
 
-            if (DBG) logdl("[addSubInfoRecord]- info size=" + sSlotIdxToSubId.size());
+            if (DBG) logdl("[addSubInfoRecord]- info size=" + sSlotIndexToSubId.size());
 
         } finally {
             Binder.restoreCallingIdentity(identity);
@@ -835,10 +835,10 @@ public class SubscriptionController extends ISub.Stub {
      * @param spn spn to be included in carrier text
      * @return true if carrier text is set, false otherwise
      */
-    public boolean setPlmnSpn(int slotId, boolean showPlmn, String plmn, boolean showSpn,
+    public boolean setPlmnSpn(int slotIndex, boolean showPlmn, String plmn, boolean showSpn,
                               String spn) {
         synchronized (mLock) {
-            int subId = getSubIdUsingPhoneId(slotId);
+            int subId = getSubIdUsingPhoneId(slotIndex);
             if (mContext.getPackageManager().resolveContentProvider(
                     SubscriptionManager.CONTENT_URI.getAuthority(), 0) == null ||
                     !SubscriptionManager.isValidSubscriptionId(subId)) {
@@ -1092,37 +1092,37 @@ public class SubscriptionController extends ISub.Stub {
     }
 
     @Override
-    public int getSlotId(int subId) {
-        if (VDBG) printStackTrace("[getSlotId] subId=" + subId);
+    public int getSlotIndex(int subId) {
+        if (VDBG) printStackTrace("[getSlotIndex] subId=" + subId);
 
         if (subId == SubscriptionManager.DEFAULT_SUBSCRIPTION_ID) {
             subId = getDefaultSubId();
         }
         if (!SubscriptionManager.isValidSubscriptionId(subId)) {
-            if (DBG) logd("[getSlotId]- subId invalid");
+            if (DBG) logd("[getSlotIndex]- subId invalid");
             return SubscriptionManager.INVALID_SIM_SLOT_INDEX;
         }
 
-        int size = sSlotIdxToSubId.size();
+        int size = sSlotIndexToSubId.size();
 
         if (size == 0)
         {
-            if (DBG) logd("[getSlotId]- size == 0, return SIM_NOT_INSERTED instead");
+            if (DBG) logd("[getSlotIndex]- size == 0, return SIM_NOT_INSERTED instead");
             return SubscriptionManager.SIM_NOT_INSERTED;
         }
 
-        for (Entry<Integer, Integer> entry: sSlotIdxToSubId.entrySet()) {
+        for (Entry<Integer, Integer> entry: sSlotIndexToSubId.entrySet()) {
             int sim = entry.getKey();
             int sub = entry.getValue();
 
             if (subId == sub)
             {
-                if (VDBG) logv("[getSlotId]- return = " + sim);
+                if (VDBG) logv("[getSlotIndex]- return = " + sim);
                 return sim;
             }
         }
 
-        if (DBG) logd("[getSlotId]- return fail");
+        if (DBG) logd("[getSlotIndex]- return fail");
         return SubscriptionManager.INVALID_SIM_SLOT_INDEX;
     }
 
@@ -1132,41 +1132,41 @@ public class SubscriptionController extends ISub.Stub {
      */
     @Override
     @Deprecated
-    public int[] getSubId(int slotIdx) {
-        if (VDBG) printStackTrace("[getSubId]+ slotIdx=" + slotIdx);
+    public int[] getSubId(int slotIndex) {
+        if (VDBG) printStackTrace("[getSubId]+ slotIndex=" + slotIndex);
 
-        // Map default slotIdx to the current default subId.
+        // Map default slotIndex to the current default subId.
         // TODO: Not used anywhere sp consider deleting as it's somewhat nebulous
         // as a slot maybe used for multiple different type of "connections"
         // such as: voice, data and sms. But we're doing the best we can and using
         // getDefaultSubId which makes a best guess.
-        if (slotIdx == SubscriptionManager.DEFAULT_SIM_SLOT_INDEX) {
-            slotIdx = getSlotId(getDefaultSubId());
-            if (VDBG) logd("[getSubId] map default slotIdx=" + slotIdx);
+        if (slotIndex == SubscriptionManager.DEFAULT_SIM_SLOT_INDEX) {
+            slotIndex = getSlotIndex(getDefaultSubId());
+            if (VDBG) logd("[getSubId] map default slotIndex=" + slotIndex);
         }
 
-        // Check that we have a valid SlotIdx
-        if (!SubscriptionManager.isValidSlotId(slotIdx)) {
-            if (DBG) logd("[getSubId]- invalid slotIdx=" + slotIdx);
+        // Check that we have a valid slotIndex
+        if (!SubscriptionManager.isValidSlotIndex(slotIndex)) {
+            if (DBG) logd("[getSubId]- invalid slotIndex=" + slotIndex);
             return null;
         }
 
-        // Check if we've got any SubscriptionInfo records using slotIdToSubId as a surrogate.
-        int size = sSlotIdxToSubId.size();
+        // Check if we've got any SubscriptionInfo records using slotIndexToSubId as a surrogate.
+        int size = sSlotIndexToSubId.size();
         if (size == 0) {
             if (VDBG) {
-                logd("[getSubId]- sSlotIdxToSubId.size == 0, return DummySubIds slotIdx="
-                        + slotIdx);
+                logd("[getSubId]- sSlotIndexToSubId.size == 0, return DummySubIds slotIndex="
+                        + slotIndex);
             }
-            return getDummySubIds(slotIdx);
+            return getDummySubIds(slotIndex);
         }
 
         // Create an array of subIds that are in this slot?
         ArrayList<Integer> subIds = new ArrayList<Integer>();
-        for (Entry<Integer, Integer> entry: sSlotIdxToSubId.entrySet()) {
+        for (Entry<Integer, Integer> entry: sSlotIndexToSubId.entrySet()) {
             int slot = entry.getKey();
             int sub = entry.getValue();
-            if (slotIdx == slot) {
+            if (slotIndex == slot) {
                 subIds.add(sub);
             }
         }
@@ -1181,8 +1181,8 @@ public class SubscriptionController extends ISub.Stub {
             if (VDBG) logd("[getSubId]- subIdArr=" + subIdArr);
             return subIdArr;
         } else {
-            if (DBG) logd("[getSubId]- numSubIds == 0, return DummySubIds slotIdx=" + slotIdx);
-            return getDummySubIds(slotIdx);
+            if (DBG) logd("[getSubId]- numSubIds == 0, return DummySubIds slotIndex=" + slotIndex);
+            return getDummySubIds(slotIndex);
         }
     }
 
@@ -1204,15 +1204,15 @@ public class SubscriptionController extends ISub.Stub {
             return SubscriptionManager.INVALID_PHONE_INDEX;
         }
 
-        int size = sSlotIdxToSubId.size();
+        int size = sSlotIndexToSubId.size();
         if (size == 0) {
             phoneId = mDefaultPhoneId;
             if (DBG) logdl("[getPhoneId]- no sims, returning default phoneId=" + phoneId);
             return phoneId;
         }
 
-        // FIXME: Assumes phoneId == slotId
-        for (Entry<Integer, Integer> entry: sSlotIdxToSubId.entrySet()) {
+        // FIXME: Assumes phoneId == slotIndex
+        for (Entry<Integer, Integer> entry: sSlotIndexToSubId.entrySet()) {
             int sim = entry.getKey();
             int sub = entry.getValue();
 
@@ -1230,7 +1230,7 @@ public class SubscriptionController extends ISub.Stub {
 
     }
 
-    private int[] getDummySubIds(int slotIdx) {
+    private int[] getDummySubIds(int slotIndex) {
         // FIXME: Remove notion of Dummy SUBSCRIPTION_ID.
         // I tested this returning null as no one appears to care,
         // but no connection came up on sprout with two sims.
@@ -1239,10 +1239,10 @@ public class SubscriptionController extends ISub.Stub {
         if (numSubs > 0) {
             int[] dummyValues = new int[numSubs];
             for (int i = 0; i < numSubs; i++) {
-                dummyValues[i] = SubscriptionManager.DUMMY_SUBSCRIPTION_ID_BASE - slotIdx;
+                dummyValues[i] = SubscriptionManager.DUMMY_SUBSCRIPTION_ID_BASE - slotIndex;
             }
             if (VDBG) {
-                logd("getDummySubIds: slotIdx=" + slotIdx
+                logd("getDummySubIds: slotIndex=" + slotIndex
                     + " return " + numSubs + " DummySubIds with each subId=" + dummyValues[0]);
             }
             return dummyValues;
@@ -1261,14 +1261,14 @@ public class SubscriptionController extends ISub.Stub {
         // Now that all security checks passes, perform the operation as ourselves.
         final long identity = Binder.clearCallingIdentity();
         try {
-            int size = sSlotIdxToSubId.size();
+            int size = sSlotIndexToSubId.size();
 
             if (size == 0) {
                 if (DBG) logdl("[clearSubInfo]- no simInfo size=" + size);
                 return 0;
             }
 
-            sSlotIdxToSubId.clear();
+            sSlotIndexToSubId.clear();
             if (DBG) logdl("[clearSubInfo]- clear size=" + size);
             return size;
         } finally {
@@ -1554,7 +1554,7 @@ public class SubscriptionController extends ISub.Stub {
         return true;
     }
 
-    // FIXME: We need we should not be assuming phoneId == slotId as it will not be true
+    // FIXME: We need we should not be assuming phoneId == slotIndex as it will not be true
     // when there are multiple subscriptions per sim and probably for other reasons.
     public int getSubIdUsingPhoneId(int phoneId) {
         int[] subIds = getSubId(phoneId);
@@ -1564,33 +1564,33 @@ public class SubscriptionController extends ISub.Stub {
         return subIds[0];
     }
 
-    public List<SubscriptionInfo> getSubInfoUsingSlotIdWithCheck(int slotId, boolean needCheck,
-            String callingPackage) {
-        if (DBG) logd("[getSubInfoUsingSlotIdWithCheck]+ slotId:" + slotId);
-
-        if (!canReadPhoneState(callingPackage, "getSubInfoUsingSlotIdWithCheck")) {
+    public List<SubscriptionInfo> getSubInfoUsingSlotIndexWithCheck(int slotIndex,
+                                                                    boolean needCheck,
+                                                                    String callingPackage) {
+        if (DBG) logd("[getSubInfoUsingSlotIndexWithCheck]+ slotIndex:" + slotIndex);
+        if (!canReadPhoneState(callingPackage, "getSubInfoUsingSlotIndexWithCheck")) {
             return null;
         }
 
         // Now that all security checks passes, perform the operation as ourselves.
         final long identity = Binder.clearCallingIdentity();
         try {
-            if (slotId == SubscriptionManager.DEFAULT_SIM_SLOT_INDEX) {
-                slotId = getSlotId(getDefaultSubId());
+            if (slotIndex == SubscriptionManager.DEFAULT_SIM_SLOT_INDEX) {
+                slotIndex = getSlotIndex(getDefaultSubId());
             }
-            if (!SubscriptionManager.isValidSlotId(slotId)) {
-                if (DBG) logd("[getSubInfoUsingSlotIdWithCheck]- invalid slotId");
+            if (!SubscriptionManager.isValidSlotIndex(slotIndex)) {
+                if (DBG) logd("[getSubInfoUsingSlotIndexWithCheck]- invalid slotIndex");
                 return null;
             }
 
             if (needCheck && !isSubInfoReady()) {
-                if (DBG) logd("[getSubInfoUsingSlotIdWithCheck]- not ready");
+                if (DBG) logd("[getSubInfoUsingSlotIndexWithCheck]- not ready");
                 return null;
             }
 
             Cursor cursor = mContext.getContentResolver().query(SubscriptionManager.CONTENT_URI,
                     null, SubscriptionManager.SIM_SLOT_INDEX + "=?",
-                    new String[]{String.valueOf(slotId)}, null);
+                    new String[]{String.valueOf(slotIndex)}, null);
             ArrayList<SubscriptionInfo> subList = null;
             try {
                 if (cursor != null) {
@@ -1609,7 +1609,7 @@ public class SubscriptionController extends ISub.Stub {
                     cursor.close();
                 }
             }
-            if (DBG) logd("[getSubInfoUsingSlotId]- null info return");
+            if (DBG) logd("[getSubInfoUsingSlotIndex]- null info return");
 
             return subList;
         } finally {
@@ -1635,7 +1635,7 @@ public class SubscriptionController extends ISub.Stub {
      */
     @Override
     public int[] getActiveSubIdList() {
-        Set<Entry<Integer, Integer>> simInfoSet = sSlotIdxToSubId.entrySet();
+        Set<Entry<Integer, Integer>> simInfoSet = sSlotIndexToSubId.entrySet();
 
         int[] subIdArr = new int[simInfoSet.size()];
         int i = 0;
@@ -1655,25 +1655,25 @@ public class SubscriptionController extends ISub.Stub {
     @Override
     public boolean isActiveSubId(int subId) {
         boolean retVal = SubscriptionManager.isValidSubscriptionId(subId)
-                && sSlotIdxToSubId.containsValue(subId);
+                && sSlotIndexToSubId.containsValue(subId);
 
         if (VDBG) logdl("[isActiveSubId]- " + retVal);
         return retVal;
     }
 
     /**
-     * Get the SIM state for the slot idx
+     * Get the SIM state for the slot index
      * @return SIM state as the ordinal of {@See IccCardConstants.State}
      */
     @Override
-    public int getSimStateForSlotIdx(int slotIdx) {
+    public int getSimStateForSlotIndex(int slotIndex) {
         State simState;
         String err;
-        if (slotIdx < 0) {
+        if (slotIndex < 0) {
             simState = IccCardConstants.State.UNKNOWN;
-            err = "invalid slotIdx";
+            err = "invalid slotIndex";
         } else {
-            Phone phone = PhoneFactory.getPhone(slotIdx);
+            Phone phone = PhoneFactory.getPhone(slotIndex);
             if (phone == null) {
                 simState = IccCardConstants.State.UNKNOWN;
                 err = "phone == null";
@@ -1689,8 +1689,8 @@ public class SubscriptionController extends ISub.Stub {
             }
         }
         if (VDBG) {
-            logd("getSimStateForSlotIdx: " + err + " simState=" + simState
-                    + " ordinal=" + simState.ordinal() + " slotIdx=" + slotIdx);
+            logd("getSimStateForSlotIndex: " + err + " simState=" + simState
+                    + " ordinal=" + simState.ordinal() + " slotIndex=" + slotIndex);
         }
         return simState.ordinal();
     }
@@ -1743,7 +1743,7 @@ public class SubscriptionController extends ISub.Stub {
      */
     @Override
     public String getSubscriptionProperty(int subId, String propKey, String callingPackage) {
-        if (!canReadPhoneState(callingPackage, "getSubInfoUsingSlotIdWithCheck")) {
+        if (!canReadPhoneState(callingPackage, "getSubInfoUsingSlotIndexWithCheck")) {
             return null;
         }
         String resultValue = null;
@@ -1823,8 +1823,8 @@ public class SubscriptionController extends ISub.Stub {
                     .from(mContext).getDefaultSmsPhoneId());
             pw.flush();
 
-            for (Entry<Integer, Integer> entry : sSlotIdxToSubId.entrySet()) {
-                pw.println(" sSlotIdxToSubId[" + entry.getKey() + "]: subId=" + entry.getValue());
+            for (Entry<Integer, Integer> entry : sSlotIndexToSubId.entrySet()) {
+                pw.println(" sSlotIndexToSubId[" + entry.getKey() + "]: subId=" + entry.getValue());
             }
             pw.flush();
             pw.println("++++++++++++++++++++++++++++++++");
index e031ffc..370834f 100644 (file)
@@ -539,7 +539,7 @@ public class SubscriptionInfoUpdater extends Handler {
         for (int i = 0; i < PROJECT_SIM_NUM; i++) {
             oldIccId[i] = null;
             List<SubscriptionInfo> oldSubInfo =
-                    SubscriptionController.getInstance().getSubInfoUsingSlotIdWithCheck(i, false,
+                    SubscriptionController.getInstance().getSubInfoUsingSlotIndexWithCheck(i, false,
                     mContext.getOpPackageName());
             if (oldSubInfo != null) {
                 oldIccId[i] = oldSubInfo.get(0).getIccId();
index 5807d85..a242de4 100644 (file)
@@ -258,7 +258,7 @@ public class CatService extends Handler implements AppInterface {
             mHandlerThread = null;
             removeCallbacksAndMessages(null);
             if (sInstance != null) {
-                if (SubscriptionManager.isValidSlotId(mSlotId)) {
+                if (SubscriptionManager.isValidSlotIndex(mSlotId)) {
                     sInstance[mSlotId] = null;
                 } else {
                     CatLog.d(this, "error: invaild slot id: " + mSlotId);
@@ -777,7 +777,7 @@ public class CatService extends Handler implements AppInterface {
         int slotId = PhoneConstants.DEFAULT_CARD_INDEX;
         SubscriptionController sControl = SubscriptionController.getInstance();
         if (sControl != null) {
-            slotId = sControl.getSlotId(sControl.getDefaultSubId());
+            slotId = sControl.getSlotIndex(sControl.getDefaultSubId());
         }
         return getInstance(null, null, null, slotId);
     }
index f74cf94..2f790b7 100644 (file)
@@ -164,9 +164,9 @@ public class ImsResolver implements ImsServiceController.ImsServiceControllerCal
          */
         int getSubId(int slotId);
         /**
-         * Mock-able interface for {@link SubscriptionManager#getSlotId(int)} used for testing.
+         * Mock-able interface for {@link SubscriptionManager#getSlotIndex(int)} used for testing.
          */
-        int getSlotId(int subId);
+        int getSlotIndex(int subId);
     }
 
     private SubscriptionManagerProxy mSubscriptionManagerProxy = new SubscriptionManagerProxy() {
@@ -181,8 +181,8 @@ public class ImsResolver implements ImsServiceController.ImsServiceControllerCal
         }
 
         @Override
-        public int getSlotId(int subId) {
-            return SubscriptionManager.getSlotId(subId);
+        public int getSlotIndex(int subId) {
+            return SubscriptionManager.getSlotIndex(subId);
         }
     };
 
@@ -610,7 +610,7 @@ public class ImsResolver implements ImsServiceController.ImsServiceControllerCal
     }
 
     private void updateBoundCarrierServices(int subId) {
-        int slotId = mSubscriptionManagerProxy.getSlotId(subId);
+        int slotId = mSubscriptionManagerProxy.getSlotIndex(subId);
         String newPackageName = mCarrierConfigManager.getConfigForSubId(subId).getString(
                 CarrierConfigManager.KEY_CONFIG_IMS_PACKAGE_OVERRIDE_STRING, null);
         if (slotId != SubscriptionManager.INVALID_SIM_SLOT_INDEX && slotId < mNumSlots) {
index 335c25e..1749768 100644 (file)
@@ -462,7 +462,7 @@ public class IccCardProxy extends Handler implements IccCard {
 
     private void broadcastIccStateChangedIntent(String value, String reason) {
         synchronized (mLock) {
-            if (mPhoneId == null || !SubscriptionManager.isValidSlotId(mPhoneId)) {
+            if (mPhoneId == null || !SubscriptionManager.isValidSlotIndex(mPhoneId)) {
                 loge("broadcastIccStateChangedIntent: mPhoneId=" + mPhoneId
                         + " is invalid; Return!!");
                 return;
@@ -512,7 +512,7 @@ public class IccCardProxy extends Handler implements IccCard {
 
     private void setExternalState(State newState, boolean override) {
         synchronized (mLock) {
-            if (mPhoneId == null || !SubscriptionManager.isValidSlotId(mPhoneId)) {
+            if (mPhoneId == null || !SubscriptionManager.isValidSlotIndex(mPhoneId)) {
                 loge("setExternalState: mPhoneId=" + mPhoneId + " is invalid; Return!!");
                 return;
             }
index 7ef441e..568238f 100644 (file)
@@ -201,7 +201,7 @@ public class SubscriptionControllerTest extends TelephonyTest {
         for (int i = 0; i < mSubList.size(); i++) {
             assertTrue(SubscriptionManager.isValidSubscriptionId(
                     mSubList.get(i).getSubscriptionId()));
-            assertTrue(SubscriptionManager.isValidSlotId(mSubList.get(i).getSimSlotIndex()));
+            assertTrue(SubscriptionManager.isValidSlotIndex(mSubList.get(i).getSimSlotIndex()));
         }
     }
 
@@ -247,7 +247,7 @@ public class SubscriptionControllerTest extends TelephonyTest {
         mSubscriptionControllerUT.clearSubInfo();
         assertFalse(mSubscriptionControllerUT.isActiveSubId(0));
         assertEquals(SubscriptionManager.SIM_NOT_INSERTED,
-                mSubscriptionControllerUT.getSlotId(0));
+                mSubscriptionControllerUT.getSlotIndex(0));
     }
 
     @Test @SmallTest
index 4bc3eea..a4fd2a7 100644 (file)
@@ -144,7 +144,7 @@ public class SubscriptionInfoUpdaterTest extends TelephonyTest {
     @SmallTest
     public void testSimAbsent() {
         doReturn(Arrays.asList(mSubInfo)).when(mSubscriptionController)
-                .getSubInfoUsingSlotIdWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
+                .getSubInfoUsingSlotIndexWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
         Intent mIntent = new Intent(TelephonyIntents.ACTION_SIM_STATE_CHANGED);
         mIntent.putExtra(IccCardConstants.INTENT_KEY_ICC_STATE,
                 IccCardConstants.INTENT_VALUE_ICC_ABSENT);
@@ -223,7 +223,7 @@ public class SubscriptionInfoUpdaterTest extends TelephonyTest {
     public void testSimLoaded() {
         /* mock new sim got loaded and there is no sim loaded before */
         doReturn(null).when(mSubscriptionController)
-                .getSubInfoUsingSlotIdWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
+                .getSubInfoUsingSlotIndexWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
         doReturn("89012604200000000000").when(mIccRecord).getIccId();
         doReturn(FAKE_MCC_MNC_1).when(mTelephonyManager).getSimOperatorNumeric(FAKE_SUB_ID_1);
         Intent intentInternalSimStateChanged =
@@ -290,7 +290,7 @@ public class SubscriptionInfoUpdaterTest extends TelephonyTest {
     public void testSimLoadedEmptyOperatorNumeric() {
         /* mock new sim got loaded and there is no sim loaded before */
         doReturn(null).when(mSubscriptionController)
-                .getSubInfoUsingSlotIdWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
+                .getSubInfoUsingSlotIndexWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
         doReturn("89012604200000000000").when(mIccRecord).getIccId();
         // operator numeric is empty
         doReturn("").when(mTelephonyManager).getSimOperatorNumeric(FAKE_SUB_ID_1);
@@ -331,7 +331,7 @@ public class SubscriptionInfoUpdaterTest extends TelephonyTest {
         }).when(mIccFileHandler).loadEFTransparent(anyInt(), any(Message.class));
 
         doReturn(Arrays.asList(mSubInfo)).when(mSubscriptionController)
-                .getSubInfoUsingSlotIdWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
+                .getSubInfoUsingSlotIndexWithCheck(eq(FAKE_SUB_ID_1), anyBoolean(), anyString());
 
         Intent mIntent = new Intent(IccCardProxy.ACTION_INTERNAL_SIM_STATE_CHANGED);
         mIntent.putExtra(IccCardConstants.INTENT_KEY_ICC_STATE,
@@ -380,7 +380,7 @@ public class SubscriptionInfoUpdaterTest extends TelephonyTest {
         doReturn(FAKE_MCC_MNC_2).when(mTelephonyManager).getSimOperatorNumeric(eq(FAKE_SUB_ID_2));
         // Mock there is no sim inserted before
         doReturn(null).when(mSubscriptionController)
-                .getSubInfoUsingSlotIdWithCheck(anyInt(), anyBoolean(), anyString());
+                .getSubInfoUsingSlotIndexWithCheck(anyInt(), anyBoolean(), anyString());
         doReturn("89012604200000000000").when(mIccRecord).getIccId();
 
         // Mock sending a sim loaded for SIM 1
index 5315c9a..5576227 100644 (file)
@@ -787,7 +787,7 @@ public class ImsResolverTest extends ImsTestBase {
             mCarrierConfigs[i] = new PersistableBundle();
             when(mMockCarrierConfigManager.getConfigForSubId(eq(i))).thenReturn(
                     mCarrierConfigs[i]);
-            when(mTestSubscriptionManagerProxy.getSlotId(eq(i))).thenReturn(i);
+            when(mTestSubscriptionManagerProxy.getSlotIndex(eq(i))).thenReturn(i);
             when(mTestSubscriptionManagerProxy.getSubId(eq(i))).thenReturn(i);
         }
 
index 5bb3275..60995a1 100644 (file)
@@ -44,7 +44,7 @@ import java.util.List;
 public class SubscriptionControllerMock extends SubscriptionController {
     final AtomicInteger mDefaultDataSubId = new AtomicInteger(INVALID_SUBSCRIPTION_ID);
     final ITelephonyRegistry.Stub mTelephonyRegistry;
-    final int[][] mSlotIdxToSubId;
+    final int[][] mSlotIndexToSubId;
 
     public static SubscriptionController init(Phone phone) {
         throw new RuntimeException("not implemented");
@@ -59,10 +59,10 @@ public class SubscriptionControllerMock extends SubscriptionController {
     public SubscriptionControllerMock(Context c, ITelephonyRegistry.Stub tr, int phoneCount) {
         super(c);
         mTelephonyRegistry = tr;
-        mSlotIdxToSubId = new int[phoneCount][];
+        mSlotIndexToSubId = new int[phoneCount][];
         for (int i = 0; i < phoneCount; i++) {
-            mSlotIdxToSubId[i] = new int[1];
-            mSlotIdxToSubId[i][0] = INVALID_SUBSCRIPTION_ID;
+            mSlotIndexToSubId[i] = new int[1];
+            mSlotIndexToSubId[i][0] = INVALID_SUBSCRIPTION_ID;
         }
     }
 
@@ -117,7 +117,7 @@ public class SubscriptionControllerMock extends SubscriptionController {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public SubscriptionInfo getActiveSubscriptionInfoForSimSlotIndex(int slotIdx, String cp){
+    public SubscriptionInfo getActiveSubscriptionInfoForSimSlotIndex(int slotIndex, String cp){
         throw new RuntimeException("not implemented");
     }
     @Override
@@ -141,11 +141,11 @@ public class SubscriptionControllerMock extends SubscriptionController {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public int addSubInfoRecord(String iccId, int slotId) {
+    public int addSubInfoRecord(String iccId, int slotIndex) {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public boolean setPlmnSpn(int slotId, boolean showPlmn, String plmn, boolean showSpn,
+    public boolean setPlmnSpn(int slotIndex, boolean showPlmn, String plmn, boolean showSpn,
             String spn) {
         throw new RuntimeException("not implemented");
     }
@@ -174,28 +174,28 @@ public class SubscriptionControllerMock extends SubscriptionController {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public int getSlotId(int subId) {
+    public int getSlotIndex(int subId) {
         throw new RuntimeException("not implemented");
     }
 
-    private boolean isInvalidSlotId(int slotIdx) {
-        if (slotIdx < 0 || slotIdx >= mSlotIdxToSubId.length) return true;
+    private boolean isInvalidslotIndex(int slotIndex) {
+        if (slotIndex < 0 || slotIndex >= mSlotIndexToSubId.length) return true;
         return false;
     }
 
     @Override
-    public int[] getSubId(int slotIdx) {
-        if (isInvalidSlotId(slotIdx)) {
+    public int[] getSubId(int slotIndex) {
+        if (isInvalidslotIndex(slotIndex)) {
             return null;
         }
-        return mSlotIdxToSubId[slotIdx];
+        return mSlotIndexToSubId[slotIndex];
     }
-    public void setSlotSubId(int slotIdx, int subId) {
-        if (isInvalidSlotId(slotIdx)) {
-            throw new RuntimeException("invalid slot specified" + slotIdx);
+    public void setSlotSubId(int slotIndex, int subId) {
+        if (isInvalidslotIndex(slotIndex)) {
+            throw new RuntimeException("invalid slot specified" + slotIndex);
         }
-        if (mSlotIdxToSubId[slotIdx][0] != subId) {
-            mSlotIdxToSubId[slotIdx][0] = subId;
+        if (mSlotIndexToSubId[slotIndex][0] != subId) {
+            mSlotIndexToSubId[slotIndex][0] = subId;
             try {
                 mTelephonyRegistry.notifySubscriptionInfoChanged();
             } catch (RemoteException ex) {}
@@ -209,8 +209,8 @@ public class SubscriptionControllerMock extends SubscriptionController {
 
         if (subId <= INVALID_SUBSCRIPTION_ID) return INVALID_PHONE_INDEX;
 
-        for (int i = 0; i < mSlotIdxToSubId.length; i++) {
-            if (mSlotIdxToSubId[i][0] == subId) return i;
+        for (int i = 0; i < mSlotIndexToSubId.length; i++) {
+            if (mSlotIndexToSubId[i][0] == subId) return i;
         }
         return INVALID_PHONE_INDEX;
     }
@@ -243,8 +243,8 @@ public class SubscriptionControllerMock extends SubscriptionController {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public List<SubscriptionInfo> getSubInfoUsingSlotIdWithCheck(int slotId, boolean needCheck,
-            String callingPackage) {
+    public List<SubscriptionInfo> getSubInfoUsingSlotIndexWithCheck(int slotId, boolean needCheck,
+                                                                    String callingPackage) {
         throw new RuntimeException("not implemented");
     }
     @Override
@@ -260,7 +260,7 @@ public class SubscriptionControllerMock extends SubscriptionController {
         throw new RuntimeException("not implemented");
     }
     @Override
-    public int getSimStateForSlotIdx(int slotIdx) {
+    public int getSimStateForSlotIndex(int slotIndex) {
         throw new RuntimeException("not implemented");
     }
     @Override