From 273a7d96a377b2f4c2e3601c2bfee4e6aaa65244 Mon Sep 17 00:00:00 2001 From: "Wesley.CW Wang" Date: Thu, 24 Jun 2021 19:24:39 +0800 Subject: [PATCH] Remove unnecessary converting us to ms - Pick from pagit/1951738 - The class "Estimate" has the menber estimateMillis and the unit of batteryUsageStats.getBatteryTimeRemainingMs() is millisecond, so converting from us to ms is unnecessary. Bug: 187379252 Bug: 184916537 Test: make RunSettingsRoboTests Change-Id: I2e8e03451352d7ad4cd44f72d5261dad35a81eb9 --- src/com/android/settings/fuelgauge/BatteryInfo.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/com/android/settings/fuelgauge/BatteryInfo.java b/src/com/android/settings/fuelgauge/BatteryInfo.java index f1b19322856..92151c0c58d 100644 --- a/src/com/android/settings/fuelgauge/BatteryInfo.java +++ b/src/com/android/settings/fuelgauge/BatteryInfo.java @@ -204,7 +204,7 @@ public class BatteryInfo { } final long prediction = discharging ? batteryUsageStats.getBatteryTimeRemainingMs() : 0; final Estimate estimate = new Estimate( - PowerUtil.convertUsToMs(prediction), + prediction, false, /* isBasedOnUsage */ EstimateKt.AVERAGE_TIME_TO_DISCHARGE_UNKNOWN); BatteryUtils.logRuntime(LOG_TAG, "time for regular BatteryInfo", startTime);