Merge "Fix the getForegroundActivityTotalTimeMs" into oc-dr1-dev
am: 451b7edfd7
Change-Id: I609e926c481e991d4039e42d0872ebcfd218679b
This commit is contained in:
@@ -330,7 +330,8 @@ public class BatteryUtils {
|
|||||||
long getForegroundActivityTotalTimeMs(BatteryStats.Uid uid, long rawRealtimeMs) {
|
long getForegroundActivityTotalTimeMs(BatteryStats.Uid uid, long rawRealtimeMs) {
|
||||||
final BatteryStats.Timer timer = uid.getForegroundActivityTimer();
|
final BatteryStats.Timer timer = uid.getForegroundActivityTimer();
|
||||||
if (timer != null) {
|
if (timer != null) {
|
||||||
return timer.getTotalTimeLocked(rawRealtimeMs, BatteryStats.STATS_SINCE_CHARGED);
|
return convertUsToMs(
|
||||||
|
timer.getTotalTimeLocked(rawRealtimeMs, BatteryStats.STATS_SINCE_CHARGED));
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
@@ -95,6 +95,8 @@ public class BatteryUtilsTest {
|
|||||||
@Mock
|
@Mock
|
||||||
private BatteryStats.Uid mUid;
|
private BatteryStats.Uid mUid;
|
||||||
@Mock
|
@Mock
|
||||||
|
private BatteryStats.Timer mTimer;
|
||||||
|
@Mock
|
||||||
private BatterySipper mNormalBatterySipper;
|
private BatterySipper mNormalBatterySipper;
|
||||||
@Mock
|
@Mock
|
||||||
private BatterySipper mWifiBatterySipper;
|
private BatterySipper mWifiBatterySipper;
|
||||||
@@ -358,6 +360,16 @@ public class BatteryUtilsTest {
|
|||||||
.isEqualTo(R.string.battery_abnormal_location_summary);
|
.isEqualTo(R.string.battery_abnormal_location_summary);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testGetForegroundActivityTotalTimeMs_returnMilliseconds() {
|
||||||
|
doReturn(mTimer).when(mUid).getForegroundActivityTimer();
|
||||||
|
doReturn(TIME_SINCE_LAST_FULL_CHARGE_US).when(mTimer).getTotalTimeLocked(anyLong(),
|
||||||
|
anyInt());
|
||||||
|
|
||||||
|
assertThat(mBatteryUtils.getForegroundActivityTotalTimeMs(mUid, 0)).isEqualTo(
|
||||||
|
TIME_SINCE_LAST_FULL_CHARGE_MS);
|
||||||
|
}
|
||||||
|
|
||||||
private BatterySipper createTestSmearBatterySipper(long activityTime, long topTime,
|
private BatterySipper createTestSmearBatterySipper(long activityTime, long topTime,
|
||||||
double totalPowerMah, int uidCode, boolean isUidNull) {
|
double totalPowerMah, int uidCode, boolean isUidNull) {
|
||||||
final BatterySipper sipper = mock(BatterySipper.class);
|
final BatterySipper sipper = mock(BatterySipper.class);
|
||||||
|
Reference in New Issue
Block a user