From e074da1804985fce23ccf0b069153862689b03ea Mon Sep 17 00:00:00 2001 From: Mike Ma Date: Thu, 5 Apr 2018 17:25:51 -0700 Subject: [PATCH] Fix ambient display time unit error Treated return value from mStats.getScreenDozeTime as ms, but indeed it is us. Fixes: 77655868 Test: manual Change-Id: Ic9838a2fca3f49a6f5a669bd1970dcc7560fb5e2 --- core/java/com/android/internal/os/BatteryStatsHelper.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/java/com/android/internal/os/BatteryStatsHelper.java b/core/java/com/android/internal/os/BatteryStatsHelper.java index 1e5bd1894843d..b49aaced83a3c 100644 --- a/core/java/com/android/internal/os/BatteryStatsHelper.java +++ b/core/java/com/android/internal/os/BatteryStatsHelper.java @@ -657,7 +657,7 @@ public class BatteryStatsHelper { * {@link #removeHiddenBatterySippers(List)}. */ private void addAmbientDisplayUsage() { - long ambientDisplayMs = mStats.getScreenDozeTime(mRawRealtimeUs, mStatsType); + long ambientDisplayMs = mStats.getScreenDozeTime(mRawRealtimeUs, mStatsType) / 1000; double power = mPowerProfile.getAveragePower(PowerProfile.POWER_AMBIENT_DISPLAY) * ambientDisplayMs / (60 * 60 * 1000); if (power > 0) {