Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1367)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java

Issue 2351113005: [Reland] Refactor WebappRegistry into a singleton instance. (Closed)
Patch Set: Comments Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/ServiceTabLauncher.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java b/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
index f1609df31ec8653afeb50f57a6384c72d498c323..8dc2a08d323d2e817ec04ef56b377a40b7566d23 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java
@@ -48,6 +48,7 @@ import org.chromium.chrome.browser.preferences.privacy.PrivacyPreferencesManager
import org.chromium.chrome.browser.share.ShareHelper;
import org.chromium.chrome.browser.webapps.ChromeWebApkHost;
import org.chromium.chrome.browser.webapps.WebApkVersionManager;
+import org.chromium.chrome.browser.webapps.WebappRegistry;
import org.chromium.content.browser.ChildProcessLauncher;
import java.util.ArrayList;
@@ -173,6 +174,10 @@ public class DeferredStartupHandler {
mDeferredTasks.add(new Runnable() {
@Override
public void run() {
+ // Initialize the WebappRegistry if it's not already initialized. Must be done on
+ // the main thread.
+ WebappRegistry.getInstance();
+
// Punt all tasks that may block on disk off onto a background thread.
initAsyncDiskTask();
@@ -276,11 +281,21 @@ public class DeferredStartupHandler {
SystemClock.uptimeMillis() - asyncTaskStartTime,
TimeUnit.MILLISECONDS);
+ // Warm up all web app shared prefs. This must be run after the WebappRegistry
+ // instance is initialized.
+ WebappRegistry.warmUpSharedPrefs();
+
return null;
} finally {
TraceEvent.end("ChromeBrowserInitializer.onDeferredStartup.doInBackground");
}
}
+
+ @Override
+ protected void onPostExecute(Void params) {
+ // Must be run on the UI thread after the WebappRegistry has been completely warmed.
+ WebappRegistry.getInstance().unregisterOldWebapps(System.currentTimeMillis());
+ }
}.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR);
}
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/ServiceTabLauncher.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698