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

Unified Diff: chrome/browser/extensions/app_host/app_host.rc

Issue 11359133: Add default icon to app_host.exe, and use it in shortcuts during installation. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 1 month 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/browser/extensions/app_host/app_host.rc
diff --git a/chrome/browser/extensions/app_host/app_host.rc b/chrome/browser/extensions/app_host/app_host.rc
index ffd5341a22e1a90bac562d7967c0a89fe017b83b..5ff0fe343dc391e1f834967fa7a080df68c6d53a 100644
--- a/chrome/browser/extensions/app_host/app_host.rc
+++ b/chrome/browser/extensions/app_host/app_host.rc
@@ -24,6 +24,22 @@ LANGUAGE LANG_ENGLISH, SUBLANG_ENGLISH_US
/////////////////////////////////////////////////////////////////////////////
//
+// Icon
+//
+
+// The application icon should: have the lowest ID, be placed first, and its
+// resource name should be alphabetically less than the name of any other icon
+// resource (http://msdn.microsoft.com/en-us/library/ms997538.aspx).
+// Note: chrome/installer/util/shell_util.cc depends on the order and number of
erikwright (departed) 2012/11/09 21:46:25 does the shell_util.cc comment apply to this, or o
huangs 2012/11/12 20:00:30 This should apply in general; icon index 0 is the
+// icons. To avoid breaking existing shortcuts, add new icons at the end
+// (following the ordering described above).
+
+// Icon for the App Lancher.
+IDR_MAINFRAME ICON "..\\..\\..\\app\\theme\\app_list.ico"
+
+
+/////////////////////////////////////////////////////////////////////////////
+//
// GOOGLEUPDATEAPPLICATIONCOMMANDS is a "well-known" marker resource defined
// by Omaha. Executables must both be signed by Google and contain the marker
// in order to expose application commands.

Powered by Google App Engine
This is Rietveld 408576698