diff --git a/src/com/android/settings/network/CarrierConfigChangedReceiver.java b/src/com/android/settings/network/CarrierConfigChangedReceiver.java index d9ff03efb6a..8a6d47d96fc 100644 --- a/src/com/android/settings/network/CarrierConfigChangedReceiver.java +++ b/src/com/android/settings/network/CarrierConfigChangedReceiver.java @@ -32,20 +32,9 @@ public class CarrierConfigChangedReceiver extends BroadcastReceiver { CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED; private final CountDownLatch mLatch; - private final boolean mIsWaitingForValidSubId; - /** - * This is the CarrierConfigChanged receiver. If it receives the carrier config changed, then it - * call the CountDownLatch.countDown(). - * If the "isWaitingForValidSubId" is true, then the receiver skip the carrier config changed - * with the subId = -1. The receiver executes the countDown when the CarrierConfigChanged - * with valid subId. - * If the "isWaitingForValidSubId" is false, then the receiver executes the countDown when - * receiving any CarrierConfigChanged. - */ - public CarrierConfigChangedReceiver(CountDownLatch latch, boolean isWaitingForValidSubId) { + public CarrierConfigChangedReceiver(CountDownLatch latch) { mLatch = latch; - mIsWaitingForValidSubId = isWaitingForValidSubId; } public void registerOn(Context context) { @@ -64,8 +53,7 @@ public class CarrierConfigChangedReceiver extends BroadcastReceiver { } private void checkSubscriptionIndex(Intent intent) { - if (intent.hasExtra(CarrierConfigManager.EXTRA_SUBSCRIPTION_INDEX) - || !mIsWaitingForValidSubId) { + if (intent.hasExtra(CarrierConfigManager.EXTRA_SUBSCRIPTION_INDEX)) { int subId = intent.getIntExtra(CarrierConfigManager.EXTRA_SUBSCRIPTION_INDEX, -1); Log.i(TAG, "subId from config changed: " + subId); mLatch.countDown(); diff --git a/src/com/android/settings/network/UiccSlotUtil.java b/src/com/android/settings/network/UiccSlotUtil.java index 7d8295b7822..7ba2e0f8ed0 100644 --- a/src/com/android/settings/network/UiccSlotUtil.java +++ b/src/com/android/settings/network/UiccSlotUtil.java @@ -141,8 +141,7 @@ public class UiccSlotUtil { inactiveRemovableSlot, /*removable sim's port Id*/ TelephonyManager.DEFAULT_PORT_INDEX, excludedLogicalSlotIndex), - context, - /*isWaitingForValidSubId=*/ true); + context); } /** @@ -180,8 +179,7 @@ public class UiccSlotUtil { performSwitchToSlot(telMgr, prepareUiccSlotMappings(uiccSlotMappings, /*slot is not psim*/ false, physicalSlotId, port, excludedLogicalSlotIndex), - context, - /*isWaitingForValidSubId=*/ false); + context); } /** @@ -233,8 +231,7 @@ public class UiccSlotUtil { } private static void performSwitchToSlot(TelephonyManager telMgr, - Collection uiccSlotMappings, Context context, - boolean isWaitingForValidSubId) + Collection uiccSlotMappings, Context context) throws UiccSlotsException { CarrierConfigChangedReceiver receiver = null; long waitingTimeMillis = @@ -244,7 +241,7 @@ public class UiccSlotUtil { DEFAULT_WAIT_AFTER_SWITCH_TIMEOUT_MILLIS); try { CountDownLatch latch = new CountDownLatch(1); - receiver = new CarrierConfigChangedReceiver(latch, isWaitingForValidSubId); + receiver = new CarrierConfigChangedReceiver(latch); receiver.registerOn(context); telMgr.setSimSlotMapping(uiccSlotMappings); latch.await(waitingTimeMillis, TimeUnit.MILLISECONDS);