From 769d16ed65258846e670f34ff341927d092cffa7 Mon Sep 17 00:00:00 2001 From: Daniel Nishi Date: Tue, 10 Jan 2017 14:22:12 -0800 Subject: [PATCH] Update StorageSettings to use StorageDashbordFragment. It previously was set to use the PrivateVolumeSettings. By swapping it with the new view, we can retain the existing behavior for external storage (PublicVolumeSettings). This also restores the summary text to the main Settings screen. Bug: 34197071, 34195303 Test: Verified locally that StorageDashboardFragment is started instead of the PrivateVolumeSettings. Change-Id: I5de0ea8de799d758272a1406856ae7de9b6fcc8e --- AndroidManifest.xml | 4 ++-- .../settings/deviceinfo/StorageDashboardFragment.java | 3 ++- src/com/android/settings/deviceinfo/StorageSettings.java | 7 ++----- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/AndroidManifest.xml b/AndroidManifest.xml index 776c4ec56ce..f529debae5e 100644 --- a/AndroidManifest.xml +++ b/AndroidManifest.xml @@ -3092,7 +3092,7 @@ @@ -3100,7 +3100,7 @@ + android:value="com.android.settings.deviceinfo.StorageSettings" /> diff --git a/src/com/android/settings/deviceinfo/StorageDashboardFragment.java b/src/com/android/settings/deviceinfo/StorageDashboardFragment.java index 5d1ac4a759a..98659d7f7f0 100644 --- a/src/com/android/settings/deviceinfo/StorageDashboardFragment.java +++ b/src/com/android/settings/deviceinfo/StorageDashboardFragment.java @@ -61,7 +61,8 @@ public class StorageDashboardFragment extends DashboardFragment { // Initialize the storage sizes that we can quickly calc. StorageManager sm = context.getSystemService(StorageManager.class); - mVolume = sm.findVolumeById(VolumeInfo.ID_PRIVATE_INTERNAL); + String volumeId = getArguments().getString(VolumeInfo.EXTRA_VOLUME_ID); + mVolume = sm.findVolumeById(volumeId); if (!isVolumeValid()) { getActivity().finish(); return; diff --git a/src/com/android/settings/deviceinfo/StorageSettings.java b/src/com/android/settings/deviceinfo/StorageSettings.java index 5ee39833a09..53d4c85b6ac 100644 --- a/src/com/android/settings/deviceinfo/StorageSettings.java +++ b/src/com/android/settings/deviceinfo/StorageSettings.java @@ -235,9 +235,8 @@ public class StorageSettings extends SettingsPreferenceFragment implements Index // Only showing primary internal storage, so just shortcut final Bundle args = new Bundle(); args.putString(VolumeInfo.EXTRA_VOLUME_ID, VolumeInfo.ID_PRIVATE_INTERNAL); - PrivateVolumeSettings.setVolumeSize(args, sTotalInternalStorage); Intent intent = Utils.onBuildStartFragmentIntent(getActivity(), - PrivateVolumeSettings.class.getName(), args, null, R.string.apps_storage, null, + StorageDashboardFragment.class.getName(), args, null, R.string.apps_storage, null, false); intent.putExtra(SettingsDrawerActivity.EXTRA_SHOW_MENU, true); getActivity().startActivity(intent); @@ -280,9 +279,7 @@ public class StorageSettings extends SettingsPreferenceFragment implements Index if (vol.getType() == VolumeInfo.TYPE_PRIVATE) { final Bundle args = new Bundle(); args.putString(VolumeInfo.EXTRA_VOLUME_ID, vol.getId()); - PrivateVolumeSettings.setVolumeSize(args, PrivateStorageInfo.getTotalSize(vol, - sTotalInternalStorage)); - startFragment(this, PrivateVolumeSettings.class.getCanonicalName(), + startFragment(this, StorageDashboardFragment.class.getCanonicalName(), -1, 0, args); return true;