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

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

Issue 1286973003: webapps: introduce helper class to store extended set of data (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compile Created 5 years, 4 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 128138290de6f34fdc95594345517f268ce4af3d..1786dadeccbbf178d5a835e898190ed60b04bc3f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
@@ -17,6 +17,7 @@ import org.chromium.base.VisibleForTesting;
import org.chromium.base.annotations.CalledByNative;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.tab.Tab;
+import org.chromium.chrome.browser.webapps.WebappDataStorage;
import org.chromium.chrome.browser.webapps.WebappLauncherActivity;
import org.chromium.content_public.browser.WebContents;
import org.chromium.content_public.common.ScreenOrientationConstants;
@@ -173,17 +174,21 @@ public class ShortcutHelper {
encodedIcon = Base64.encodeToString(byteArray, Base64.DEFAULT);
}
+ String uuid = UUID.randomUUID().toString();
+
// Add the shortcut as a launcher icon for a full-screen Activity.
shortcutIntent = new Intent();
shortcutIntent.setAction(sDelegate.getFullscreenAction());
shortcutIntent.putExtra(EXTRA_ICON, encodedIcon);
- shortcutIntent.putExtra(EXTRA_ID, UUID.randomUUID().toString());
+ shortcutIntent.putExtra(EXTRA_ID, uuid);
shortcutIntent.putExtra(EXTRA_NAME, name);
shortcutIntent.putExtra(EXTRA_SHORT_NAME, shortName);
shortcutIntent.putExtra(EXTRA_URL, url);
shortcutIntent.putExtra(EXTRA_ORIENTATION, orientation);
shortcutIntent.putExtra(EXTRA_MAC, getEncodedMac(context, url));
shortcutIntent.putExtra(EXTRA_THEME_COLOR, themeColor);
+
+ WebappDataStorage.open(context, uuid).putSplashIcon(icon);
} else {
// Add the shortcut as a launcher icon to open in the browser Activity.
shortcutIntent = BookmarkUtils.createShortcutIntent(url);

Powered by Google App Engine
This is Rietveld 408576698