From 21a0bed00c3ea157ef0fff91417a5023824835d3 Mon Sep 17 00:00:00 2001 From: ykhung Date: Wed, 16 Feb 2022 12:04:27 +0800 Subject: [PATCH] Add log for shouldShowBatteryAttributionList() checking Bug: 195306545 Test: make SettingsRoboTests Change-Id: I0f41d87731a6703f453a9b0476a7534b22e53884 --- .../BatteryAppListPreferenceController.java | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java index 815f2fd2e52..8dbce7979db 100644 --- a/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java +++ b/src/com/android/settings/fuelgauge/BatteryAppListPreferenceController.java @@ -36,6 +36,7 @@ import android.text.TextUtils; import android.text.format.DateUtils; import android.util.ArrayMap; import android.util.SparseArray; +import android.util.Log; import androidx.annotation.VisibleForTesting; import androidx.preference.Preference; @@ -64,6 +65,7 @@ import java.util.List; */ public class BatteryAppListPreferenceController extends AbstractPreferenceController implements PreferenceControllerMixin, LifecycleObserver, OnPause, OnDestroy { + private static final String TAG = "BatteryAppListPreferenceController"; @VisibleForTesting static final boolean USE_FAKE_DATA = false; private static final int MAX_ITEMS_TO_LIST = USE_FAKE_DATA ? 30 : 20; @@ -103,9 +105,14 @@ public class BatteryAppListPreferenceController extends AbstractPreferenceContro PowerProfile powerProfile = new PowerProfile(context); // Cheap hack to try to figure out if the power_profile.xml was populated. - return powerProfile.getAveragePowerForOrdinal( - PowerProfile.POWER_GROUP_DISPLAY_SCREEN_FULL, 0) - >= MIN_AVERAGE_POWER_THRESHOLD_MILLI_AMP; + final double averagePowerForOrdinal = powerProfile.getAveragePowerForOrdinal( + PowerProfile.POWER_GROUP_DISPLAY_SCREEN_FULL, 0); + final boolean shouldShowBatteryAttributionList = + averagePowerForOrdinal >= MIN_AVERAGE_POWER_THRESHOLD_MILLI_AMP; + if (!shouldShowBatteryAttributionList) { + Log.w(TAG, "shouldShowBatteryAttributionList(): " + averagePowerForOrdinal); + } + return shouldShowBatteryAttributionList; } };