Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 42151571 authored by Narayan Kamath's avatar Narayan Kamath
Browse files

BatteryStats: Fix bogus calls to addHistoryRecordLocked. 

The second argument (byte cmd) was being cast to long and being used
as uptimeMs.

Test: BatteryStatsTests
Bug: 71355449
Change-Id: If40745d25038cd5d206244f7a28d6902bbb43c18
parent d11dc140

core/java/com/android/internal/os/BatteryStatsImpl.java

+2 −2
Original line number Diff line number Diff line
@@ -3743,7 +3743,7 @@ public class BatteryStatsImpl extends BatteryStats { 














        if (mNumHistoryItems == MAX_HISTORY_ITEMS















                || mNumHistoryItems == MAX_MAX_HISTORY_ITEMS) {













            addHistoryRecordLocked(elapsedRealtimeMs, HistoryItem.CMD_OVERFLOW);













            addHistoryBufferLocked(elapsedRealtimeMs, uptimeMs, HistoryItem.CMD_OVERFLOW, cur);















        }





























        if (mNumHistoryItems >= MAX_HISTORY_ITEMS) {









@@ -3760,7 +3760,7 @@ public class BatteryStatsImpl extends BatteryStats {













            }















        }



























        addHistoryRecordLocked(elapsedRealtimeMs, HistoryItem.CMD_UPDATE);













        addHistoryBufferLocked(elapsedRealtimeMs, uptimeMs, HistoryItem.CMD_UPDATE, cur);















    }





























    public void addHistoryEventLocked(long elapsedRealtimeMs, long uptimeMs, int code,