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

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

Issue 1103273011: Break apart shortcut icon generation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 | « chrome/android/java/src/org/chromium/chrome/browser/BookmarkUtils.java ('k') | no next file » | 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/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 32b3a817532588582e14efa2203746466b2dae50..f5708d5fe867350a96d5ed35c820c865f7273b40 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ShortcutHelper.java
@@ -155,12 +155,13 @@ public class ShortcutHelper {
shortcutIntent.putExtra(EXTRA_MAC, getEncodedMac(context, url));
} else {
// Add the shortcut as a launcher icon to open in the browser Activity.
- shortcutIntent = BookmarkUtils.createShortcutIntent(context, url);
+ shortcutIntent = BookmarkUtils.createShortcutIntent(url);
}
shortcutIntent.setPackage(context.getPackageName());
- context.sendBroadcast(BookmarkUtils.createAddToHomeIntent(context, shortcutIntent, title,
- icon, url, red, green, blue));
+ Bitmap launchIcon = BookmarkUtils.createLauncherIcon(context, icon, url, red, green, blue);
+ context.sendBroadcast(BookmarkUtils.createAddToHomeIntent(shortcutIntent, title,
+ launchIcon, url));
// Alert the user about adding the shortcut.
if (returnToHomescreen) {
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/BookmarkUtils.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698