Fix problem of white box appearing in themes without an NTP background image....

chrome/browser/browser_theme_provider.cc

Index: chrome/browser/browser_theme_provider.cc
===================================================================
--- chrome/browser/browser_theme_provider.cc	(revision 25836)
+++ chrome/browser/browser_theme_provider.cc	(working copy)
@@ -256,6 +256,13 @@
     resource_names_[IDR_THEME_TAB_BACKGROUND] = "theme_tab_background";
     resource_names_[IDR_THEME_TAB_BACKGROUND_INCOGNITO] =
         "theme_tab_background_incognito";
+    resource_names_[IDR_THEME_TOOLBAR] = "theme_toolbar";

[Glen Murphy, 2009/09/10 04:45:36]

Are these meant to be part of this CL?

[Miranda Callahan, 2009/09/10 15:28:43]

Yes and no; when I moved the code in DOMUIThemeSource, it exposed a problem with
my change to HasCustomImage -- namely, that THEME_NTP_BACKGROUND wasn't
contained in resource_names_, and so it failed that check -- causing the
background image to be placed weirdly.  This fixes that hole -- all
themeable_images are now mapped to their names in resource_names_, and so can be
tested for customhood.

+    resource_names_[IDR_THEME_TAB_BACKGROUND_V] = "theme_tab_background_v";
+    resource_names_[IDR_THEME_NTP_BACKGROUND] = "theme_ntp_background";
+    resource_names_[IDR_THEME_BUTTON_BACKGROUND] = "theme_button_background";
+    resource_names_[IDR_THEME_NTP_ATTRIBUTION] = "theme_ntp_attribution";
+    resource_names_[IDR_THEME_WINDOW_CONTROL_BACKGROUND] =
+        "theme_window_control_background";
   }
 }