From 7d7080ab6419beb1e995a007c1270a8241bc021c Mon Sep 17 00:00:00 2001 From: jackqdyulei Date: Tue, 16 May 2017 16:39:03 -0700 Subject: [PATCH] Drop the misc and not smear it. Bug: 38347725 Test: RunSettingsRoboTests Change-Id: I573e4fb945bc212505c9211d9b92f77a6a9a2739 --- src/com/android/settings/fuelgauge/BatteryUtils.java | 5 +++-- .../src/com/android/settings/fuelgauge/BatteryUtilsTest.java | 3 +-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/com/android/settings/fuelgauge/BatteryUtils.java b/src/com/android/settings/fuelgauge/BatteryUtils.java index 3b6aa493d69..8e730c03c04 100644 --- a/src/com/android/settings/fuelgauge/BatteryUtils.java +++ b/src/com/android/settings/fuelgauge/BatteryUtils.java @@ -132,8 +132,9 @@ public class BatteryUtils { if (shouldHideSipper(sipper)) { sippers.remove(i); if (sipper.drainType != BatterySipper.DrainType.OVERCOUNTED - && sipper.drainType != BatterySipper.DrainType.SCREEN) { - // Don't add it if it is overcounted or screen + && sipper.drainType != BatterySipper.DrainType.SCREEN + && sipper.drainType != BatterySipper.DrainType.UNACCOUNTED) { + // Don't add it if it is overcounted, unaccounted or screen proportionalSmearPowerMah += sipper.totalPowerMah; } } diff --git a/tests/robotests/src/com/android/settings/fuelgauge/BatteryUtilsTest.java b/tests/robotests/src/com/android/settings/fuelgauge/BatteryUtilsTest.java index fad5c5e1d84..f6a8587f516 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/BatteryUtilsTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/BatteryUtilsTest.java @@ -192,8 +192,7 @@ public class BatteryUtilsTest { final double totalUsage = mBatteryUtils.removeHiddenBatterySippers(sippers); assertThat(sippers).containsExactly(mNormalBatterySipper); - assertThat(totalUsage).isWithin(PRECISION).of( - BATTERY_SYSTEM_USAGE + BATTERY_UNACCOUNTED_USAGE); + assertThat(totalUsage).isWithin(PRECISION).of(BATTERY_SYSTEM_USAGE); } @Test