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

Browse Source
This commit is contained in:
Siim Sammul
2022-04-29 08:09:45 +00:00
committed by Android (Google) Code Review
parent 7b5e80f2f6 1fd6b3b6b9
commit 1b81993295
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 {