Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove unnecessary +1ms to quota window calculation." into rvc-dev am: cbe9c27876 am: 207d5759f2

Browse Source 
Change-Id: I86db877e696594e854a77bea18897f63695cff5c
This commit is contained in:
TreeHugger Robot
2020-03-24 19:14:54 +00:00
committed by Automerger Merge Worker
parent 157a9c850f 207d5759f2
commit 8bba9c0a9c
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/core/java/com/android/server/AlarmManagerService.java
View File

@@ -1876,7 +1876,7 @@ class AlarmManagerService extends SystemService {
// package was t(q) then the next delivery must be after t(q) + <window_size>
final long t = mAppWakeupHistory.getNthLastWakeupForPackage(
sourcePackage, sourceUserId, quotaForBucket);
minElapsed = t + 1 + mConstants.APP_STANDBY_WINDOW;
minElapsed = t + mConstants.APP_STANDBY_WINDOW;
}
if (alarm.expectedWhenElapsed < minElapsed) {
alarm.whenElapsed = alarm.maxWhenElapsed = minElapsed;

12
services/tests/mockingservicestests/src/com/android/server/AlarmManagerServiceTest.java
View File

@@ -496,7 +496,7 @@ public class AlarmManagerServiceTest {
// This one should get deferred on set
setTestAlarm(ELAPSED_REALTIME_WAKEUP, firstTrigger + quota,
getNewMockPendingIntent());
final long expectedNextTrigger = firstTrigger + 1 + mAppStandbyWindow;
final long expectedNextTrigger = firstTrigger + mAppStandbyWindow;
assertEquals("Incorrect next alarm trigger", expectedNextTrigger, mTestTimer.getElapsed());
}
@@ -516,7 +516,7 @@ public class AlarmManagerServiceTest {
mNowElapsedTest = mTestTimer.getElapsed();
mTestTimer.expire();
}
final long expectedNextTrigger = firstTrigger + 1 + mAppStandbyWindow;
final long expectedNextTrigger = firstTrigger + mAppStandbyWindow;
assertEquals("Incorrect next alarm trigger", expectedNextTrigger, mTestTimer.getElapsed());
}
@@ -676,7 +676,7 @@ public class AlarmManagerServiceTest {
final int rareQuota = mService.getQuotaForBucketLocked(STANDBY_BUCKET_RARE);
// The last alarm should now be deferred.
final long expectedNextTrigger = (firstTrigger + workingQuota - 1 - rareQuota)
+ mAppStandbyWindow + 1;
+ mAppStandbyWindow;
assertEquals("Incorrect next alarm trigger", expectedNextTrigger, mTestTimer.getElapsed());
}
@@ -695,7 +695,7 @@ public class AlarmManagerServiceTest {
}
}
// The last alarm should be deferred due to exceeding the quota
final long deferredTrigger = firstTrigger + 1 + mAppStandbyWindow;
final long deferredTrigger = firstTrigger + mAppStandbyWindow;
assertEquals(deferredTrigger, mTestTimer.getElapsed());
// Upgrading the bucket now
@@ -730,7 +730,7 @@ public class AlarmManagerServiceTest {
mTestTimer.expire();
}
// Any subsequent alarms in queue should all be deferred
assertEquals(firstTrigger + mAppStandbyWindow + 1, mTestTimer.getElapsed());
assertEquals(firstTrigger + mAppStandbyWindow, mTestTimer.getElapsed());
// Paroling now
assertAndHandleParoleChanged(true);
@@ -744,7 +744,7 @@ public class AlarmManagerServiceTest {
assertAndHandleParoleChanged(false);
// Subsequent alarms should again get deferred
final long expectedNextTrigger = (firstTrigger + 5) + 1 + mAppStandbyWindow;
final long expectedNextTrigger = (firstTrigger + 5) + mAppStandbyWindow;
assertEquals("Incorrect next alarm trigger", expectedNextTrigger, mTestTimer.getElapsed());
}