From 1c3cb4505c1abe58f16f1f00475b5f397617a301 Mon Sep 17 00:00:00 2001 From: ykhung Date: Fri, 29 Apr 2022 10:35:39 +0800 Subject: [PATCH] Hide running time information for "Android System" entry Hide the running time information for "Android System" entry, since this entry will combine multiple system components together. It will provide incorrect running time information. The getRealUid() method maps many UIDs to Process.SYSTEM_UID(1000), which results in combining all of those UIDs into one "Android System" entry. This is the expected behavior. Bug: 220717612 Test: make RunSettingsRoboTests -j56 ROBOTEST_FILTER="com.android.settings.fuelgauge" Change-Id: I0aece0f1e86c36e55a2b5a966b4da9fdea14c748 --- .../settings/fuelgauge/BatteryAppListPreferenceController.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java index 8fb0d1e0f64..db4cc536579 100644 --- a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java +++ b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java @@ -424,6 +424,9 @@ public class BatteryAppListPreferenceController extends AbstractPreferenceContro @VisibleForTesting void setUsageSummary(Preference preference, BatteryEntry entry) { + if (entry.getUid() == Process.SYSTEM_UID) { + return; + } // Only show summary when usage time is longer than one minute final long usageTimeMs = entry.getTimeInForegroundMs(); if (shouldShowSummary(entry) && usageTimeMs >= DateUtils.MINUTE_IN_MILLIS) {