From 2102faf3e3e10ade12fd105aa19fec7522ec6c89 Mon Sep 17 00:00:00 2001 From: Joel Galenson Date: Tue, 16 Apr 2019 15:44:52 -0700 Subject: [PATCH] Do not special case the storage permission group. We are now not showing storage usages in Permission Controller, including in the API Settings uses. We can thus remove this code so that Settings will reflect what Permissions Hub shows. Bug: 130567526 Test: See the same graph here and in the dashboard. Change-Id: Iaa7d95b05f4321543ed42833ac3a7babdb9914f0 --- .../privacy/PermissionBarChartPreferenceController.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/com/android/settings/privacy/PermissionBarChartPreferenceController.java b/src/com/android/settings/privacy/PermissionBarChartPreferenceController.java index d8c66e4feb3..399216c3466 100644 --- a/src/com/android/settings/privacy/PermissionBarChartPreferenceController.java +++ b/src/com/android/settings/privacy/PermissionBarChartPreferenceController.java @@ -176,9 +176,6 @@ public class PermissionBarChartPreferenceController extends BasePreferenceContro return null; } - // STOPSHIP: Ignore the STORAGE group since it's going away. - usageInfos.removeIf(usage -> usage.getName().equals("android.permission-group.STORAGE")); - final BarViewInfo[] barViewInfos = new BarViewInfo[ Math.min(BarChartPreference.MAXIMUM_BAR_VIEWS, usageInfos.size())];