PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Revert "Revert "[Settings] Code Refactor - ActiveSubscip..."

Browse Source 
Revert submission 15102785-Fix build break of refactor ActiveSubscriptionsListener

Reason for revert: Build breakage.
Reverted Changes:
I9ae819f06:Revert "Revert "[Settings] Code Refactor - ActiveS...
I15819472b:[Settings] Fix build breakage

Change-Id: Ib830b7b00504264a703a03a00bfdfcf65135d394
This commit is contained in:
Bonian Chen
2021-06-28 05:23:33 +00:00
parent c437177d4b
commit 69fc8e0ccd
3 changed files with 19 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/com/android/settings/network/ActiveSubscriptionsListener.java → src/com/android/settings/network/ActiveSubsciptionsListener.java
View File

@@ -39,7 +39,7 @@ import java.util.concurrent.atomic.AtomicInteger;
/** /**
* A listener for active subscription change * A listener for active subscription change
*/ */
public abstract class ActiveSubscriptionsListener public abstract class ActiveSubsciptionsListener
extends SubscriptionManager.OnSubscriptionsChangedListener extends SubscriptionManager.OnSubscriptionsChangedListener
implements AutoCloseable { implements AutoCloseable {
@@ -73,7 +73,7 @@ public abstract class ActiveSubscriptionsListener
* @param looper {@code Looper} of this listener * @param looper {@code Looper} of this listener
* @param context {@code Context} of this listener * @param context {@code Context} of this listener
*/ */
public ActiveSubscriptionsListener(Looper looper, Context context) { public ActiveSubsciptionsListener(Looper looper, Context context) {
this(looper, context, SubscriptionManager.INVALID_SUBSCRIPTION_ID); this(looper, context, SubscriptionManager.INVALID_SUBSCRIPTION_ID);
} }
@@ -84,7 +84,7 @@ public abstract class ActiveSubscriptionsListener
* @param context {@code Context} of this listener * @param context {@code Context} of this listener
* @param subscriptionId for subscription on this listener * @param subscriptionId for subscription on this listener
*/ */
public ActiveSubscriptionsListener(Looper looper, Context context, int subscriptionId) { public ActiveSubsciptionsListener(Looper looper, Context context, int subscriptionId) {
super(looper); super(looper);
mLooper = looper; mLooper = looper;
mContext = context; mContext = context;

22
src/com/android/settings/network/telephony/MobileNetworkSettings.java
View File

@@ -37,7 +37,7 @@ import androidx.preference.Preference;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.datausage.BillingCyclePreferenceController; import com.android.settings.datausage.BillingCyclePreferenceController;
import com.android.settings.datausage.DataUsageSummaryPreferenceController; import com.android.settings.datausage.DataUsageSummaryPreferenceController;
import com.android.settings.network.ActiveSubscriptionsListener; import com.android.settings.network.ActiveSubsciptionsListener;
import com.android.settings.network.CarrierWifiTogglePreferenceController; import com.android.settings.network.CarrierWifiTogglePreferenceController;
import com.android.settings.network.telephony.cdma.CdmaSubscriptionPreferenceController; import com.android.settings.network.telephony.cdma.CdmaSubscriptionPreferenceController;
import com.android.settings.network.telephony.cdma.CdmaSystemSelectPreferenceController; import com.android.settings.network.telephony.cdma.CdmaSystemSelectPreferenceController;
@@ -74,9 +74,9 @@ public class MobileNetworkSettings extends AbstractMobileNetworkSettings {
private UserManager mUserManager; private UserManager mUserManager;
private String mClickedPrefKey; private String mClickedPrefKey;
private ActiveSubscriptionsListener mActiveSubscriptionsListener; private ActiveSubsciptionsListener mActiveSubsciptionsListener;
private boolean mDropFirstSubscriptionChangeNotify; private boolean mDropFirstSubscriptionChangeNotify;
private int mActiveSubscriptionsListenerCount; private int mActiveSubsciptionsListenerCount;
public MobileNetworkSettings() { public MobileNetworkSettings() {
super(UserManager.DISALLOW_CONFIG_MOBILE_NETWORKS); super(UserManager.DISALLOW_CONFIG_MOBILE_NETWORKS);
@@ -218,8 +218,8 @@ public class MobileNetworkSettings extends AbstractMobileNetworkSettings {
super.onResume(); super.onResume();
// TODO: remove log after fixing b/182326102 // TODO: remove log after fixing b/182326102
Log.d(LOG_TAG, "onResume() subId=" + mSubId); Log.d(LOG_TAG, "onResume() subId=" + mSubId);
if (mActiveSubscriptionsListener == null) { if (mActiveSubsciptionsListener == null) {
mActiveSubscriptionsListener = new ActiveSubscriptionsListener( mActiveSubsciptionsListener = new ActiveSubsciptionsListener(
getContext().getMainLooper(), getContext(), mSubId) { getContext().getMainLooper(), getContext(), mSubId) {
public void onChanged() { public void onChanged() {
onSubscriptionDetailChanged(); onSubscriptionDetailChanged();
@@ -227,7 +227,7 @@ public class MobileNetworkSettings extends AbstractMobileNetworkSettings {
}; };
mDropFirstSubscriptionChangeNotify = true; mDropFirstSubscriptionChangeNotify = true;
} }
mActiveSubscriptionsListener.start(); mActiveSubsciptionsListener.start();
} }
private void onSubscriptionDetailChanged() { private void onSubscriptionDetailChanged() {
@@ -236,21 +236,21 @@ public class MobileNetworkSettings extends AbstractMobileNetworkSettings {
Log.d(LOG_TAG, "Callback during onResume()"); Log.d(LOG_TAG, "Callback during onResume()");
return; return;
} }
mActiveSubscriptionsListenerCount++; mActiveSubsciptionsListenerCount++;
if (mActiveSubscriptionsListenerCount != 1) { if (mActiveSubsciptionsListenerCount != 1) {
return; return;
} }
ThreadUtils.postOnMainThread(() -> { ThreadUtils.postOnMainThread(() -> {
mActiveSubscriptionsListenerCount = 0; mActiveSubsciptionsListenerCount = 0;
redrawPreferenceControllers(); redrawPreferenceControllers();
}); });
} }
@Override @Override
public void onDestroy() { public void onDestroy() {
if (mActiveSubscriptionsListener != null) { if (mActiveSubsciptionsListener != null) {
mActiveSubscriptionsListener.stop(); mActiveSubsciptionsListener.stop();
} }
super.onDestroy(); super.onDestroy();
} }

10
tests/unit/src/com/android/settings/network/ActiveSubscriptionsListenerTest.java → tests/unit/src/com/android/settings/network/ActiveSubsciptionsListenerTest.java
View File

@@ -49,7 +49,7 @@ import java.util.ArrayList;
import java.util.List; import java.util.List;
@RunWith(AndroidJUnit4.class) @RunWith(AndroidJUnit4.class)
public class ActiveSubscriptionsListenerTest { public class ActiveSubsciptionsListenerTest {
private static final int SUB_ID1 = 3; private static final int SUB_ID1 = 3;
private static final int SUB_ID2 = 7; private static final int SUB_ID2 = 7;
@@ -68,7 +68,7 @@ public class ActiveSubscriptionsListenerTest {
private SubscriptionManager mSubscriptionManager; private SubscriptionManager mSubscriptionManager;
private List<SubscriptionInfo> mActiveSubscriptions; private List<SubscriptionInfo> mActiveSubscriptions;
private ActiveSubscriptionsListenerImpl mListener; private ActiveSubsciptionsListenerImpl mListener;
private BroadcastReceiver mReceiver; private BroadcastReceiver mReceiver;
@Before @Before
@@ -83,7 +83,7 @@ public class ActiveSubscriptionsListenerTest {
addMockSubscription(SUB_ID2); addMockSubscription(SUB_ID2);
doReturn(mActiveSubscriptions).when(mSubscriptionManager).getActiveSubscriptionInfoList(); doReturn(mActiveSubscriptions).when(mSubscriptionManager).getActiveSubscriptionInfoList();
mListener = spy(new ActiveSubscriptionsListenerImpl(Looper.getMainLooper(), mContext)); mListener = spy(new ActiveSubsciptionsListenerImpl(Looper.getMainLooper(), mContext));
doReturn(mSubscriptionManager).when(mListener).getSubscriptionManager(); doReturn(mSubscriptionManager).when(mListener).getSubscriptionManager();
mReceiver = mListener.getSubscriptionChangeReceiver(); mReceiver = mListener.getSubscriptionChangeReceiver();
} }
@@ -99,8 +99,8 @@ public class ActiveSubscriptionsListenerTest {
mListener.close(); mListener.close();
} }
public class ActiveSubscriptionsListenerImpl extends ActiveSubscriptionsListener { public class ActiveSubsciptionsListenerImpl extends ActiveSubsciptionsListener {
public ActiveSubscriptionsListenerImpl(Looper looper, Context context) { public ActiveSubsciptionsListenerImpl(Looper looper, Context context) {
super(looper, context); super(looper, context);
} }