From 129c98bba1d2fcd055d1be443c0f46ea30898ca8 Mon Sep 17 00:00:00 2001 From: Raff Tsai Date: Mon, 30 Sep 2019 18:28:16 +0800 Subject: [PATCH] Remove summary loader in some fragments - Summary Loader only works in injected items, but those fragments are not injected to Settings UI. The SummaryLoader class them are useless. Bug: 141653158 Test: robolectric Change-Id: I1ca0bf5a1f417b7fe86b28e96aff7d4aee2cf2d3 --- AndroidManifest.xml | 3 - .../settings/applications/ProcessStatsSummary.java | 38 ----------- .../manageapplications/ManageApplications.java | 36 ---------- .../settings/datausage/DataUsageSummary.java | 77 ---------------------- src/com/android/settings/wifi/WifiSettings.java | 38 ----------- src/com/android/settings/wifi/WifiSettings2.java | 38 ----------- .../settings/datausage/DataUsageSummaryTest.java | 18 ----- 7 files changed, 248 deletions(-) diff --git a/AndroidManifest.xml b/AndroidManifest.xml index f9dc83b658..310af0e6e5 100644 --- a/AndroidManifest.xml +++ b/AndroidManifest.xml @@ -2344,9 +2344,6 @@ - - - diff --git a/src/com/android/settings/applications/ProcessStatsSummary.java b/src/com/android/settings/applications/ProcessStatsSummary.java index e5760651b6..a9f4ade704 100644 --- a/src/com/android/settings/applications/ProcessStatsSummary.java +++ b/src/com/android/settings/applications/ProcessStatsSummary.java @@ -15,7 +15,6 @@ */ package com.android.settings.applications; -import android.app.Activity; import android.app.settings.SettingsEnums; import android.content.Context; import android.os.Bundle; @@ -30,7 +29,6 @@ import com.android.settings.SummaryPreference; import com.android.settings.Utils; import com.android.settings.applications.ProcStatsData.MemInfo; import com.android.settings.core.SubSettingLauncher; -import com.android.settings.dashboard.SummaryLoader; public class ProcessStatsSummary extends ProcessStatsBase implements OnPreferenceClickListener { @@ -127,40 +125,4 @@ public class ProcessStatsSummary extends ProcessStatsBase implements OnPreferenc } return false; } - - private static class SummaryProvider implements SummaryLoader.SummaryProvider { - - private final Context mContext; - private final SummaryLoader mSummaryLoader; - - public SummaryProvider(Context context, SummaryLoader summaryLoader) { - mContext = context; - mSummaryLoader = summaryLoader; - } - - @Override - public void setListening(boolean listening) { - if (listening) { - ProcStatsData statsManager = new ProcStatsData(mContext, false); - statsManager.setDuration(sDurations[0]); - MemInfo memInfo = statsManager.getMemInfo(); - String usedResult = Formatter.formatShortFileSize(mContext, - (long) memInfo.realUsedRam); - String totalResult = Formatter.formatShortFileSize(mContext, - (long) memInfo.realTotalRam); - mSummaryLoader.setSummary(this, mContext.getString(R.string.memory_summary, - usedResult, totalResult)); - } - } - } - - public static final SummaryLoader.SummaryProviderFactory SUMMARY_PROVIDER_FACTORY - = new SummaryLoader.SummaryProviderFactory() { - @Override - public SummaryLoader.SummaryProvider createSummaryProvider(Activity activity, - SummaryLoader summaryLoader) { - return new SummaryProvider(activity, summaryLoader); - } - }; - } diff --git a/src/com/android/settings/applications/manageapplications/ManageApplications.java b/src/com/android/settings/applications/manageapplications/ManageApplications.java index 734552e22c..90ab71a151 100644 --- a/src/com/android/settings/applications/manageapplications/ManageApplications.java +++ b/src/com/android/settings/applications/manageapplications/ManageApplications.java @@ -92,7 +92,6 @@ import com.android.settings.applications.AppStateUsageBridge; import com.android.settings.applications.AppStateUsageBridge.UsageState; import com.android.settings.applications.AppStateWriteSettingsBridge; import com.android.settings.applications.AppStorageSettings; -import com.android.settings.applications.InstalledAppCounter; import com.android.settings.applications.UsageAccessDetails; import com.android.settings.applications.appinfo.AppInfoDashboardFragment; import com.android.settings.applications.appinfo.DrawOverlayDetails; @@ -100,7 +99,6 @@ import com.android.settings.applications.appinfo.ExternalSourcesDetails; import com.android.settings.applications.appinfo.WriteSettingsDetails; import com.android.settings.core.InstrumentedFragment; import com.android.settings.core.SubSettingLauncher; -import com.android.settings.dashboard.SummaryLoader; import com.android.settings.fuelgauge.HighPowerDetail; import com.android.settings.notification.AppNotificationSettings; import com.android.settings.notification.ConfigureNotificationSettings; @@ -1518,38 +1516,4 @@ public class ManageApplications extends InstrumentedFragment } } } - - private static class SummaryProvider implements SummaryLoader.SummaryProvider { - - private final Context mContext; - private final SummaryLoader mLoader; - - private SummaryProvider(Context context, SummaryLoader loader) { - mContext = context; - mLoader = loader; - } - - @Override - public void setListening(boolean listening) { - if (listening) { - new InstalledAppCounter(mContext, InstalledAppCounter.IGNORE_INSTALL_REASON, - mContext.getPackageManager()) { - @Override - protected void onCountComplete(int num) { - mLoader.setSummary(SummaryProvider.this, - mContext.getString(R.string.apps_summary, num)); - } - }.execute(); - } - } - } - - public static final SummaryLoader.SummaryProviderFactory SUMMARY_PROVIDER_FACTORY - = new SummaryLoader.SummaryProviderFactory() { - @Override - public SummaryLoader.SummaryProvider createSummaryProvider(Activity activity, - SummaryLoader summaryLoader) { - return new SummaryProvider(activity, summaryLoader); - } - }; } diff --git a/src/com/android/settings/datausage/DataUsageSummary.java b/src/com/android/settings/datausage/DataUsageSummary.java index 6597ecb4c8..d26fc307c3 100644 --- a/src/com/android/settings/datausage/DataUsageSummary.java +++ b/src/com/android/settings/datausage/DataUsageSummary.java @@ -21,7 +21,6 @@ import android.net.NetworkTemplate; import android.os.Bundle; import android.telephony.SubscriptionInfo; import android.telephony.SubscriptionManager; -import android.telephony.SubscriptionPlan; import android.text.BidiFormatter; import android.text.Spannable; import android.text.SpannableString; @@ -34,11 +33,8 @@ import androidx.preference.Preference; import androidx.preference.PreferenceScreen; import com.android.settings.R; -import com.android.settings.Utils; -import com.android.settings.dashboard.SummaryLoader; import com.android.settingslib.NetworkPolicyEditor; import com.android.settingslib.core.AbstractPreferenceController; -import com.android.settingslib.net.DataUsageController; import java.util.ArrayList; import java.util.List; @@ -250,77 +246,4 @@ public class DataUsageSummary extends DataUsageBaseFragment implements DataUsage updateState(); mSummaryController.updateState(mSummaryPreference); } - - private static class SummaryProvider - implements SummaryLoader.SummaryProvider { - - private final Activity mActivity; - private final SummaryLoader mSummaryLoader; - private final DataUsageController mDataController; - - public SummaryProvider(Activity activity, SummaryLoader summaryLoader) { - mActivity = activity; - mSummaryLoader = summaryLoader; - mDataController = new DataUsageController(activity); - } - - @Override - public void setListening(boolean listening) { - if (listening) { - if (DataUsageUtils.hasSim(mActivity)) { - mSummaryLoader.setSummary(this, - mActivity.getString(R.string.data_usage_summary_format, - formatUsedData())); - } else { - final DataUsageController.DataUsageInfo info = - mDataController.getWifiDataUsageInfo(); - - if (info == null) { - mSummaryLoader.setSummary(this, null); - } else { - final CharSequence wifiFormat = mActivity - .getText(R.string.data_usage_wifi_format); - final CharSequence sizeText = - DataUsageUtils.formatDataUsage(mActivity, info.usageLevel); - mSummaryLoader.setSummary(this, - TextUtils.expandTemplate(wifiFormat, sizeText)); - } - } - } - } - - private CharSequence formatUsedData() { - SubscriptionManager subscriptionManager = (SubscriptionManager) mActivity - .getSystemService(Context.TELEPHONY_SUBSCRIPTION_SERVICE); - int defaultSubId = subscriptionManager.getDefaultSubscriptionId(); - if (defaultSubId == SubscriptionManager.INVALID_SUBSCRIPTION_ID) { - return formatFallbackData(); - } - SubscriptionPlan dfltPlan = DataUsageSummaryPreferenceController - .getPrimaryPlan(subscriptionManager, defaultSubId); - if (dfltPlan == null) { - return formatFallbackData(); - } - if (DataUsageSummaryPreferenceController.unlimited(dfltPlan.getDataLimitBytes())) { - return DataUsageUtils.formatDataUsage(mActivity, dfltPlan.getDataUsageBytes()); - } else { - return Utils.formatPercentage(dfltPlan.getDataUsageBytes(), - dfltPlan.getDataLimitBytes()); - } - } - - private CharSequence formatFallbackData() { - DataUsageController.DataUsageInfo info = mDataController.getDataUsageInfo(); - if (info == null) { - return DataUsageUtils.formatDataUsage(mActivity, 0); - } else if (info.limitLevel <= 0) { - return DataUsageUtils.formatDataUsage(mActivity, info.usageLevel); - } else { - return Utils.formatPercentage(info.usageLevel, info.limitLevel); - } - } - } - - public static final SummaryLoader.SummaryProviderFactory SUMMARY_PROVIDER_FACTORY - = SummaryProvider::new; } diff --git a/src/com/android/settings/wifi/WifiSettings.java b/src/com/android/settings/wifi/WifiSettings.java index ec799de26a..2741139111 100644 --- a/src/com/android/settings/wifi/WifiSettings.java +++ b/src/com/android/settings/wifi/WifiSettings.java @@ -60,12 +60,10 @@ import com.android.settings.RestrictedSettingsFragment; import com.android.settings.SettingsActivity; import com.android.settings.core.FeatureFlags; import com.android.settings.core.SubSettingLauncher; -import com.android.settings.dashboard.SummaryLoader; import com.android.settings.datausage.DataUsagePreference; import com.android.settings.datausage.DataUsageUtils; import com.android.settings.location.ScanningSettings; import com.android.settings.search.BaseSearchIndexProvider; -import com.android.settings.widget.SummaryUpdater.OnSummaryChangeListener; import com.android.settings.widget.SwitchBarController; import com.android.settings.wifi.details.WifiNetworkDetailsFragment; import com.android.settings.wifi.dpp.WifiDppUtils; @@ -1228,42 +1226,6 @@ public class WifiSettings extends RestrictedSettingsFragment } }; - private static class SummaryProvider - implements SummaryLoader.SummaryProvider, OnSummaryChangeListener { - - private final Context mContext; - private final SummaryLoader mSummaryLoader; - - @VisibleForTesting - WifiSummaryUpdater mSummaryHelper; - - public SummaryProvider(Context context, SummaryLoader summaryLoader) { - mContext = context; - mSummaryLoader = summaryLoader; - mSummaryHelper = new WifiSummaryUpdater(mContext, this); - } - - - @Override - public void setListening(boolean listening) { - mSummaryHelper.register(listening); - } - - @Override - public void onSummaryChanged(String summary) { - mSummaryLoader.setSummary(this, summary); - } - } - - public static final SummaryLoader.SummaryProviderFactory SUMMARY_PROVIDER_FACTORY - = new SummaryLoader.SummaryProviderFactory() { - @Override - public SummaryLoader.SummaryProvider createSummaryProvider(Activity activity, - SummaryLoader summaryLoader) { - return new SummaryProvider(activity, summaryLoader); - } - }; - private void handleConfigNetworkSubmitEvent(Intent data) { final WifiConfiguration wifiConfiguration = data.getParcelableExtra( ConfigureAccessPointFragment.NETWORK_CONFIG_KEY); diff --git a/src/com/android/settings/wifi/WifiSettings2.java b/src/com/android/settings/wifi/WifiSettings2.java index ed697edcc2..58745e83c1 100644 --- a/src/com/android/settings/wifi/WifiSettings2.java +++ b/src/com/android/settings/wifi/WifiSettings2.java @@ -54,12 +54,10 @@ import com.android.settings.R; import com.android.settings.RestrictedSettingsFragment; import com.android.settings.SettingsActivity; import com.android.settings.core.SubSettingLauncher; -import com.android.settings.dashboard.SummaryLoader; import com.android.settings.datausage.DataUsagePreference; import com.android.settings.datausage.DataUsageUtils; import com.android.settings.location.ScanningSettings; import com.android.settings.search.BaseSearchIndexProvider; -import com.android.settings.widget.SummaryUpdater.OnSummaryChangeListener; import com.android.settings.widget.SwitchBarController; import com.android.settingslib.search.Indexable; import com.android.settingslib.search.SearchIndexable; @@ -676,40 +674,4 @@ public class WifiSettings2 extends RestrictedSettingsFragment return result; } }; - - private static class SummaryProvider - implements SummaryLoader.SummaryProvider, OnSummaryChangeListener { - - private final Context mContext; - private final SummaryLoader mSummaryLoader; - - @VisibleForTesting - WifiSummaryUpdater mSummaryHelper; - - public SummaryProvider(Context context, SummaryLoader summaryLoader) { - mContext = context; - mSummaryLoader = summaryLoader; - mSummaryHelper = new WifiSummaryUpdater(mContext, this); - } - - - @Override - public void setListening(boolean listening) { - mSummaryHelper.register(listening); - } - - @Override - public void onSummaryChanged(String summary) { - mSummaryLoader.setSummary(this, summary); - } - } - - public static final SummaryLoader.SummaryProviderFactory SUMMARY_PROVIDER_FACTORY - = new SummaryLoader.SummaryProviderFactory() { - @Override - public SummaryLoader.SummaryProvider createSummaryProvider(Activity activity, - SummaryLoader summaryLoader) { - return new SummaryProvider(activity, summaryLoader); - } - }; } diff --git a/tests/robotests/src/com/android/settings/datausage/DataUsageSummaryTest.java b/tests/robotests/src/com/android/settings/datausage/DataUsageSummaryTest.java index 0672bd4c61..fea1d76918 100644 --- a/tests/robotests/src/com/android/settings/datausage/DataUsageSummaryTest.java +++ b/tests/robotests/src/com/android/settings/datausage/DataUsageSummaryTest.java @@ -74,7 +74,6 @@ public class DataUsageSummaryTest { private TelephonyManager mTelephonyManager; private Context mContext; private FragmentActivity mActivity; - private SummaryLoader.SummaryProvider mSummaryProvider; /** * This set up is contrived to get a passing test so that the build doesn't block without tests. @@ -96,9 +95,6 @@ public class DataUsageSummaryTest { shadowTelephonyManager.setTelephonyManagerForSubscriptionId(1, mTelephonyManager); mActivity = spy(Robolectric.buildActivity(FragmentActivity.class).get()); doReturn(mNetworkStatsManager).when(mActivity).getSystemService(NetworkStatsManager.class); - - mSummaryProvider = DataUsageSummary.SUMMARY_PROVIDER_FACTORY - .createSummaryProvider(mActivity, mSummaryLoader); } @Test @@ -111,20 +107,6 @@ public class DataUsageSummaryTest { } @Test - public void setListening_shouldBlankSummaryWithNoSim() { - ShadowDataUsageUtils.HAS_SIM = false; - mSummaryProvider.setListening(true); - verify(mSummaryLoader).setSummary(mSummaryProvider, null); - } - - @Test - public void setListening_shouldSetSummaryWithSim() { - ShadowDataUsageUtils.HAS_SIM = true; - mSummaryProvider.setListening(true); - verify(mSummaryLoader).setSummary(anyObject(), endsWith(" of data used")); - } - - @Test @Config(shadows = ShadowSubscriptionManager.class) public void configuration_withSim_shouldShowMobileAndWifi() { ShadowDataUsageUtils.IS_MOBILE_DATA_SUPPORTED = true; -- 2.11.0