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

Commit 7c6e6e92 authored by jackqdyulei's avatar jackqdyulei
Browse files

Remove STATE_TOP_SLEEPING from foreground time 

That state means the time when app is on top while screen is off,
we should not count it as foreground usage time.

Bug: 37547898
Test: RunSettingsRoboTests
Change-Id: I67e0c135d0c4b71cd667037b0d5dc7673b53607e
parent 9e89d528

src/com/android/settings/fuelgauge/BatteryUtils.java

+0 −1
Original line number Diff line number Diff line
@@ -102,7 +102,6 @@ public class BatteryUtils { 
        final long rawRealTimeUs = convertMsToUs(SystemClock.elapsedRealtime());

        final int foregroundTypes[] = {BatteryStats.Uid.PROCESS_STATE_TOP,

                BatteryStats.Uid.PROCESS_STATE_FOREGROUND_SERVICE,

                BatteryStats.Uid.PROCESS_STATE_TOP_SLEEPING,

                BatteryStats.Uid.PROCESS_STATE_FOREGROUND};

        Log.v(TAG, "package: " + mPackageManager.getNameForUid(uid.getUid()));

tests/robotests/src/com/android/settings/fuelgauge/BatteryUtilsTest.java

+2 −2
Original line number Diff line number Diff line
@@ -63,9 +63,9 @@ public class BatteryUtilsTest { 
    private static final long TIME_STATE_BACKGROUND = 6000 * UNIT;



    private static final int UID = 123;

    private static final long TIME_EXPECTED_FOREGROUND = 9000;

    private static final long TIME_EXPECTED_FOREGROUND = 6500;

    private static final long TIME_EXPECTED_BACKGROUND = 6000;

    private static final long TIME_EXPECTED_ALL = 15000;

    private static final long TIME_EXPECTED_ALL = 12500;

    private static final double BATTERY_SCREEN_USAGE = 300;

    private static final double BATTERY_SYSTEM_USAGE = 600;

    private static final double BATTERY_OVERACCOUNTED_USAGE = 500;