PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix potential OOM problem"

Browse Source
This commit is contained in:
TreeHugger Robot
2021-12-27 02:49:05 +00:00
committed by Android (Google) Code Review
parent b67f0974f2 72f823ba9e
commit 7f04708f44
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/com/android/settings/core/instrumentation/SettingsIntelligenceLogWriter.java
View File

@@ -48,6 +48,8 @@ public class SettingsIntelligenceLogWriter implements LogWriter {
private static final String LOG = "logs"; private static final String LOG = "logs";
private static final long MESSAGE_DELAY = DateUtils.MINUTE_IN_MILLIS; // 1 minute private static final long MESSAGE_DELAY = DateUtils.MINUTE_IN_MILLIS; // 1 minute
// Based on the exp, 99.5% users collect less than 150 data in 1 minute.
private static final int CACHE_LOG_THRESHOLD = 150;
private List<SettingsLog> mSettingsLogList; private List<SettingsLog> mSettingsLogList;
private SendLogHandler mLogHandler; private SendLogHandler mLogHandler;
@@ -128,7 +130,8 @@ public class SettingsIntelligenceLogWriter implements LogWriter {
mLogHandler.post(() -> { mLogHandler.post(() -> {
mSettingsLogList.add(settingsLog); mSettingsLogList.add(settingsLog);
}); });
if (action == SettingsEnums.ACTION_CONTEXTUAL_CARD_DISMISS) { if (action == SettingsEnums.ACTION_CONTEXTUAL_CARD_DISMISS
|| mSettingsLogList.size() >= CACHE_LOG_THRESHOLD) {
// Directly send this event to notify SI instantly that the card is dismissed // Directly send this event to notify SI instantly that the card is dismissed
mLogHandler.sendLog(); mLogHandler.sendLog();
} else { } else {