Don't create unnecessary objects.

Only create a new LinkedList if one doesn't exist already for the user.

Bug: n/a
Test: atest UsageStatsTest
This commit is contained in:
Varun Shah
2020-08-06 22:09:32 +00:00
parent 3e2e74c132
commit 1c691f5bd2

View File

@@ -763,11 +763,12 @@ public class UsageStatsService extends SystemService implements
return;
}
final LinkedList<Event> events = mReportedEvents.get(userId, new LinkedList<>());
events.add(event);
if (mReportedEvents.get(userId) == null) {
LinkedList<Event> events = mReportedEvents.get(userId);
if (events == null) {
events = new LinkedList<>();
mReportedEvents.put(userId, events);
}
events.add(event);
if (events.size() == 1) {
// Every time a file is persisted to disk, mReportedEvents is cleared for this user
// so trigger a flush to disk every time the first event has been added.