From 1ade809ff92470149d39238b0b96492a71a148ac Mon Sep 17 00:00:00 2001 From: Raff Tsai Date: Fri, 8 Nov 2019 16:23:51 +0800 Subject: [PATCH] Filter invalid uid from Battery list - Normally uid is start from 0. That was a bug in framework during some time period, and was not reproduced later. We just remove this kind of invalid item from UI. Fixes: 141404334 Test: manual Change-Id: I964a8b8af18ac6a5d8d2dea89722ea466e27e6ad --- .../settings/fuelgauge/BatteryAppListPreferenceController.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java index 7741a979a92..70b964c6cf0 100644 --- a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java +++ b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java @@ -46,7 +46,6 @@ import com.android.settings.R; import com.android.settings.SettingsActivity; import com.android.settings.core.InstrumentedPreferenceFragment; import com.android.settings.core.PreferenceControllerMixin; -import com.android.settingslib.applications.AppUtils; import com.android.settingslib.core.AbstractPreferenceController; import com.android.settingslib.core.lifecycle.Lifecycle; import com.android.settingslib.core.lifecycle.LifecycleObserver; @@ -355,7 +354,7 @@ public class BatteryAppListPreferenceController extends AbstractPreferenceContro // Don't show over-counted, unaccounted and hidden system module in any condition return sipper.drainType == BatterySipper.DrainType.OVERCOUNTED || sipper.drainType == BatterySipper.DrainType.UNACCOUNTED - || mBatteryUtils.isHiddenSystemModule(sipper); + || mBatteryUtils.isHiddenSystemModule(sipper) || sipper.getUid() < 0; } @VisibleForTesting