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

Merge "Fix a bug where error count was reset before returning it." into tm-dev am: 1b81993295

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/17982495

Change-Id: I560f7f832ddc6dba034c5b8e5c5bc75a1ebe3f6e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Siim Sammul
2022-04-29 16:47:03 +00:00
committed by Automerger Merge Worker
parent b9d48538d5 1b81993295
commit 716a1dd70b
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
services/core/java/com/android/server/am/DropboxRateLimiter.java
View File

@@ -64,9 +64,10 @@ public class DropboxRateLimiter {
}
if (now - errRecord.getStartTime() > RATE_LIMIT_BUFFER_DURATION) {
final int errCount = recentlyDroppedCount(errRecord);
errRecord.setStartTime(now);
errRecord.setCount(1);
return new RateLimitResult(false, recentlyDroppedCount(errRecord));
return new RateLimitResult(false, errCount);
}
errRecord.incrementCount();

10
services/tests/servicestests/src/com/android/server/am/DropboxRateLimiterTest.java
View File

@@ -90,6 +90,16 @@ public class DropboxRateLimiterTest {
mRateLimiter.shouldRateLimit("tag", "p").droppedCountSinceRateLimitActivated());
assertEquals(2,
mRateLimiter.shouldRateLimit("tag", "p").droppedCountSinceRateLimitActivated());
// After 11 seconds the rate limiting buffer will be cleared and rate limiting will stop.
mClock.setOffsetMillis(11000);
// The first call after rate limiting stops will still return the number of dropped events.
assertEquals(2,
mRateLimiter.shouldRateLimit("tag", "p").droppedCountSinceRateLimitActivated());
// The next call should show that the dropped event counter was reset.
assertEquals(0,
mRateLimiter.shouldRateLimit("tag", "p").droppedCountSinceRateLimitActivated());
}
private static class TestClock implements DropboxRateLimiter.Clock {