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

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

Issue 2384413003: Revert of Refactor WebappRegistry into a singleton instance. (Closed)
Patch Set: 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
Index: chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
index 9e271e6fdbc34252831fd4bc896b36aa7ea2467f..b174519765b4af9ef2162054881fadbf36241fd6 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
@@ -167,7 +167,7 @@
// Store the webapp data so that it is accessible without the intent. Once this
// process is complete, call back to native code to start the splash image
// download.
- WebappRegistry.getInstance().register(
+ WebappRegistry.registerWebapp(
id, new WebappRegistry.FetchWebappDataStorageCallback() {
@Override
public void onWebappDataStorageRetrieved(WebappDataStorage storage) {
@@ -227,7 +227,7 @@
}
/**
- * Stores the specified bitmap as the splash screen for a web app.
+ * Creates a storage location and stores the data for a web app using {@link WebappDataStorage}.
* @param id ID of the web app which is storing data.
* @param splashImage Image which should be displayed on the splash screen of
* the web app. This can be null of there is no image to show.
@@ -235,8 +235,16 @@
@SuppressWarnings("unused")
@CalledByNative
private static void storeWebappSplashImage(final String id, final Bitmap splashImage) {
- WebappDataStorage storage = WebappRegistry.getInstance().getWebappDataStorage(id);
- if (storage != null) storage.updateSplashScreenImage(splashImage);
+ WebappRegistry.getWebappDataStorage(
+ id, new WebappRegistry.FetchWebappDataStorageCallback() {
+ @Override
+ public void onWebappDataStorageRetrieved(WebappDataStorage storage) {
+ if (storage == null) return;
+
+ storage.updateSplashScreenImage(splashImage);
+ }
+
+ });
}
/**

Powered by Google App Engine
This is Rietveld 408576698