diff options
author | Nate Jiang <qiangjiang@google.com> | 2020-05-05 12:28:48 -0700 |
---|---|---|
committer | Nate Jiang <qiangjiang@google.com> | 2020-05-06 19:21:56 -0700 |
commit | 62667d5e507fc676cfbe7df34bb78b7e25948d37 (patch) | |
tree | 9ff530111a13d93083c6a0cc338663ba1b117f28 /libs/WifiTrackerLib/src | |
parent | 4d68b0507ef4bede9fd8af1a93f0637e78b5fe6c (diff) | |
download | frameworks_opt_net_wifi-62667d5e507fc676cfbe7df34bb78b7e25948d37.tar.gz frameworks_opt_net_wifi-62667d5e507fc676cfbe7df34bb78b7e25948d37.tar.bz2 frameworks_opt_net_wifi-62667d5e507fc676cfbe7df34bb78b7e25948d37.zip |
Show App name when carrier id not set
Cherry Pick ag/11398998 and ag/11403417
Bug: 155697387
Test: atest WifiTrackerLibTests
Change-Id: I25a18a6a537df2de9e002a6fcdb09ba652561882
Diffstat (limited to 'libs/WifiTrackerLib/src')
-rw-r--r-- | libs/WifiTrackerLib/src/com/android/wifitrackerlib/Utils.java | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/libs/WifiTrackerLib/src/com/android/wifitrackerlib/Utils.java b/libs/WifiTrackerLib/src/com/android/wifitrackerlib/Utils.java index 78617ba5a..ee115cecc 100644 --- a/libs/WifiTrackerLib/src/com/android/wifitrackerlib/Utils.java +++ b/libs/WifiTrackerLib/src/com/android/wifitrackerlib/Utils.java @@ -616,6 +616,9 @@ class Utils { * Get the SIM carrier name for target subscription Id. */ static @Nullable String getCarrierNameForSubId(@NonNull Context context, int subId) { + if (subId == SubscriptionManager.INVALID_SUBSCRIPTION_ID) { + return null; + } TelephonyManager telephonyManager = (TelephonyManager) context.getSystemService(Context.TELEPHONY_SERVICE); if (telephonyManager == null) return null; @@ -639,9 +642,8 @@ class Utils { * Get the best match subscription Id for target WifiConfiguration. */ static int getSubIdForConfig(@NonNull Context context, @NonNull WifiConfiguration config) { - int dataSubId = SubscriptionManager.getDefaultDataSubscriptionId(); if (config.carrierId == TelephonyManager.UNKNOWN_CARRIER_ID) { - return dataSubId; + return SubscriptionManager.INVALID_SUBSCRIPTION_ID; } SubscriptionManager subscriptionManager = (SubscriptionManager) context.getSystemService( @@ -655,6 +657,7 @@ class Utils { } int matchSubId = SubscriptionManager.INVALID_SUBSCRIPTION_ID; + int dataSubId = SubscriptionManager.getDefaultDataSubscriptionId(); for (SubscriptionInfo subInfo : subInfoList) { if (subInfo.getCarrierId() == config.carrierId) { matchSubId = subInfo.getSubscriptionId(); @@ -689,8 +692,13 @@ class Utils { if (context == null || wifiConfig == null || !isSimCredential(wifiConfig)) { return ""; } - - int subId = getSubIdForConfig(context, wifiConfig); + int subId; + if (wifiConfig.carrierId == TelephonyManager.UNKNOWN_CARRIER_ID) { + // Config without carrierId use default data subscription. + subId = SubscriptionManager.getDefaultSubscriptionId(); + } else { + subId = getSubIdForConfig(context, wifiConfig); + } if (subId == SubscriptionManager.INVALID_SUBSCRIPTION_ID || isImsiPrivacyProtectionProvided(context, subId)) { return ""; |