Merge "Hide carrier service package in Wi-Fi data usage settings" into udc-dev am: 7a7193c865

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/23123005

Change-Id: Iecb665be4c15549b679aeb28ff353dc067a7190d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

src/com/android/settings/datausage/DataUsageList.java

@@ -165,11 +165,7 @@ public class DataUsageList extends DataUsageBaseFragment
         }
 
         processArgument();
-        mMobileNetworkRepository = MobileNetworkRepository.getInstance(getContext());
-        ThreadUtils.postOnBackgroundThread(() -> {
-            mSubscriptionInfoEntity = mMobileNetworkRepository.getSubInfoById(
-                    String.valueOf(mSubId));
-        });
+        updateSubscriptionInfoEntity();
         mDataStateListener = new MobileDataEnabledListener(activity, this);
     }
 
@@ -294,6 +290,15 @@ public class DataUsageList extends DataUsageBaseFragment
         }
     }
 
+    @VisibleForTesting
+    void updateSubscriptionInfoEntity() {
+        mMobileNetworkRepository = MobileNetworkRepository.getInstance(getContext());
+        ThreadUtils.postOnBackgroundThread(() -> {
+            mSubscriptionInfoEntity = mMobileNetworkRepository.getSubInfoById(
+                    String.valueOf(mSubId));
+        });
+    }
+
     /**
      * Implementation of {@code MobileDataEnabledListener.Client}
      */
@@ -503,11 +508,13 @@ public class DataUsageList extends DataUsageBaseFragment
         Collections.sort(items);
         final List<String> packageNames = Arrays.asList(getContext().getResources().getStringArray(
                 R.array.datausage_hiding_carrier_service_package_names));
+        // When there is no specified SubscriptionInfo, Wi-Fi data usage will be displayed.
+        // In this case, the carrier service package also needs to be hidden.
         boolean shouldHidePackageName = mSubscriptionInfoEntity != null
                 ? Arrays.stream(getContext().getResources().getIntArray(
                         R.array.datausage_hiding_carrier_service_carrier_id))
                 .anyMatch(carrierId -> (carrierId == mSubscriptionInfoEntity.carrierId))
-                : false;
+                : true;
 
         for (int i = 0; i < items.size(); i++) {
             UidDetail detail = mUidDetailProvider.getUidDetail(items.get(i).key, true);

tests/robotests/src/com/android/settings/datausage/DataUsageListTest.java

@@ -106,6 +106,7 @@ public class DataUsageListTest {
         ReflectionHelpers.setField(mDataUsageList, "services", mNetworkServices);
         doReturn(mLoaderManager).when(mDataUsageList).getLoaderManager();
         mDataUsageList.mLoadingViewController = mock(LoadingViewController.class);
+        doNothing().when(mDataUsageList).updateSubscriptionInfoEntity();
     }
 
     @Test