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

Unified Diff: chrome/browser/browser_theme_provider.cc

Issue 1601002: BrowserThemePack: Adds persistant ids for the reload endcaps. (Closed)
Patch Set: Update test data Created 10 years, 9 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/browser/browser_theme_provider.cc
diff --git a/chrome/browser/browser_theme_provider.cc b/chrome/browser/browser_theme_provider.cc
index d062fbc096ddbf69763b4c76858697f190634607..dab544e2f5596d8e66fdd7604d65f00c4fb75f10 100644
--- a/chrome/browser/browser_theme_provider.cc
+++ b/chrome/browser/browser_theme_provider.cc
@@ -155,7 +155,9 @@ bool HasThemeableImage(int themeable_image_id) {
// The image resources that will be tinted by the 'button' tint value.
// If you change this list, you must increment the version number in
-// browser_theme_pack.cc.
+// browser_theme_pack.cc and you should assign persistant ids to the
+// data table at the start of said file or otherwise tinted versions of
+// these resources will not be created.
const int kToolbarButtonIDs[] = {
IDR_BACK, IDR_BACK_D, IDR_BACK_H, IDR_BACK_P,
IDR_FORWARD, IDR_FORWARD_D, IDR_FORWARD_H, IDR_FORWARD_P,

Powered by Google App Engine
This is Rietveld 408576698