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

Unified Diff: chrome/app/chrome_exe.rc

Issue 13947002: Update App Launcher icons, splitting for Chromium, Chrome, Canary. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: selfnits Created 7 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
Index: chrome/app/chrome_exe.rc
diff --git a/chrome/app/chrome_exe.rc b/chrome/app/chrome_exe.rc
index d9dd131a45c7f8bf7cfe0351dd7aa676f102c77d..e0bace1e24b0ff1861c3064242c1323cd0d57e01 100644
--- a/chrome/app/chrome_exe.rc
+++ b/chrome/app/chrome_exe.rc
@@ -46,11 +46,17 @@ IDR_SXS ICON "theme\google_chrome\chrome_sxs.
#else
IDR_MAINFRAME ICON "theme\chromium\chromium.ico"
#endif
+
// Icon for the App Lancher. Named with a leading X to keep the name
// alphabetically after IDR_SXS.
-// The SXS icon must have an index of 5 for Chrome and 1 for Chromium, these
-// constants are used in Chrome code to identify it.
-IDR_X_APP_LIST ICON "theme\\app_list.ico"
+#if defined(GOOGLE_CHROME_BUILD)
+// The App Launcher icon must have an index of 5 in Chrome, and 6 for SXS.
+IDR_X_APP_LIST ICON "theme\\google_chrome\\app_list.ico"
+IDR_X_APP_LIST_SXS ICON "theme\\google_chrome\\app_list_sxs.ico"
tapted 2013/04/10 06:36:37 here's the canary :D -- just in the chrome.exe
+#else
+// The App Launcher icon must have an index of 1 for Chromium.
+IDR_X_APP_LIST ICON "theme\\chromium\\app_list.ico"
+#endif
#endif // English (U.S.) resources
/////////////////////////////////////////////////////////////////////////////

Powered by Google App Engine
This is Rietveld 408576698