From ebe36cf96c920dc00e00ea4c1e8b5b3fe19fa351 Mon Sep 17 00:00:00 2001 From: changbetty Date: Wed, 8 Jan 2020 22:06:26 +0800 Subject: [PATCH] [Mainline] To use @SystemApi to replace non-formal APIs for mainline - getResourcesForSubId(Context, int, boolean) - @SystemApi getResourcesForSubId(Context, int) - isUsableSubIdValue(int subId) - public isUsableSubscriptionId(int subId) Bug: 147206736 Test: make RunSettingsRoboTests ROBOTEST_FILTER=ResetNetworkTest make RunSettingsRoboTests ROBOTEST_FILTER=ResetNetworkConfirmTest make RunSettingsRoboTests ROBOTEST_FILTER=CellularFallbackPreferenceControllerTest make RunSettingsRoboTests ROBOTEST_FILTER=WifiCallingSettingsForSubTest Change-Id: I1a67db80b114a438afda7d45af90e54af38703c3 --- src/com/android/settings/ResetNetwork.java | 6 +++--- src/com/android/settings/ResetNetworkConfirm.java | 2 +- .../settings/wifi/CellularFallbackPreferenceController.java | 3 +-- .../settings/wifi/calling/WifiCallingSettingsForSub.java | 2 +- 4 files changed, 6 insertions(+), 7 deletions(-) diff --git a/src/com/android/settings/ResetNetwork.java b/src/com/android/settings/ResetNetwork.java index 5ac2c0bf85a..e2adcc71c22 100644 --- a/src/com/android/settings/ResetNetwork.java +++ b/src/com/android/settings/ResetNetwork.java @@ -170,13 +170,13 @@ public class ResetNetwork extends InstrumentedFragment { if (mSubscriptions != null && mSubscriptions.size() > 0) { // Get the default subscription in the order of data, voice, sms, first up. int defaultSubscription = SubscriptionManager.getDefaultDataSubscriptionId(); - if (!SubscriptionManager.isUsableSubIdValue(defaultSubscription)) { + if (!SubscriptionManager.isUsableSubscriptionId(defaultSubscription)) { defaultSubscription = SubscriptionManager.getDefaultVoiceSubscriptionId(); } - if (!SubscriptionManager.isUsableSubIdValue(defaultSubscription)) { + if (!SubscriptionManager.isUsableSubscriptionId(defaultSubscription)) { defaultSubscription = SubscriptionManager.getDefaultSmsSubscriptionId(); } - if (!SubscriptionManager.isUsableSubIdValue(defaultSubscription)) { + if (!SubscriptionManager.isUsableSubscriptionId(defaultSubscription)) { defaultSubscription = SubscriptionManager.getDefaultSubscriptionId(); } diff --git a/src/com/android/settings/ResetNetworkConfirm.java b/src/com/android/settings/ResetNetworkConfirm.java index ee11ce089c9..9f1b8fb2ab1 100644 --- a/src/com/android/settings/ResetNetworkConfirm.java +++ b/src/com/android/settings/ResetNetworkConfirm.java @@ -203,7 +203,7 @@ public class ResetNetworkConfirm extends InstrumentedFragment { private void restoreDefaultApn(Context context) { Uri uri = Uri.parse(ApnSettings.RESTORE_CARRIERS_URI); - if (SubscriptionManager.isUsableSubIdValue(mSubId)) { + if (SubscriptionManager.isUsableSubscriptionId(mSubId)) { uri = Uri.withAppendedPath(uri, "subId/" + String.valueOf(mSubId)); } diff --git a/src/com/android/settings/wifi/CellularFallbackPreferenceController.java b/src/com/android/settings/wifi/CellularFallbackPreferenceController.java index 659ddbe96bd..eab50a62e45 100644 --- a/src/com/android/settings/wifi/CellularFallbackPreferenceController.java +++ b/src/com/android/settings/wifi/CellularFallbackPreferenceController.java @@ -68,8 +68,7 @@ public class CellularFallbackPreferenceController extends TogglePreferenceContro @VisibleForTesting Resources getResourcesForSubId(int subscriptionId) { - return SubscriptionManager.getResourcesForSubId(mContext, subscriptionId, - false /* useRootLocale */); + return SubscriptionManager.getResourcesForSubId(mContext, subscriptionId); } private boolean avoidBadWifiCurrentSettings() { diff --git a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java index 5f26e2dbfbd..ea34fce100c 100644 --- a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java +++ b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java @@ -639,6 +639,6 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment @VisibleForTesting Resources getResourcesForSubId() { - return SubscriptionManager.getResourcesForSubId(getContext(), mSubId, false); + return SubscriptionManager.getResourcesForSubId(getContext(), mSubId); } }