Really theme the find bar. We were previously getting the DefaultThemeProvider (whose accessor was mistakenly named 'getDIALOGthemeprovider')

BUG=11702
TEST=Install a theme and verify that the find bar matches it.
TBR=finnur

Review URL: http://codereview.chromium.org/115729



git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17257 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 12 insertions, 6 deletions

diff --git a/views/widget/widget.h b/views/widget/widget.h
index efa16c5..14a61a0 100644
--- a/views/widget/widget.h
+++ b/views/widget/widget.h
@@ -104,10 +104,10 @@ class Widget {
   // Get the theme provider.
   virtual ThemeProvider* GetThemeProvider() const { return NULL; }
 
-  // Get the dialog theme provider; this is necessary for when a dialog has
-  // no profile (and ThemeProvider) associated with it. The dialog theme
-  // provider provides a default set of bitmaps that such dialogs can use.
-  virtual ThemeProvider* GetDialogThemeProvider() { return NULL; }
+  // Get the default theme provider; this is necessary for when a widget has
+  // no profile (and ThemeProvider) associated with it. The default theme
+  // provider provides a default set of bitmaps that such widgets can use.
+  virtual ThemeProvider* GetDefaultThemeProvider() { return NULL; }
 };
 
 }  // namespace views
diff --git a/views/widget/widget_win.cc b/views/widget/widget_win.cc
index 79ec700..2d4d3ac 100644
--- a/views/widget/widget_win.cc
+++ b/views/widget/widget_win.cc
@@ -395,8 +395,14 @@ TooltipManager* WidgetWin::GetTooltipManager() {
 
 ThemeProvider* WidgetWin::GetThemeProvider() const {
   Widget* widget = GetRootWidget();
-  if (widget) {
-    ThemeProvider* provider = widget->GetDialogThemeProvider();
+  if (widget && widget != this) {
+    // Attempt to get the theme provider, and fall back to the default theme
+    // provider if not found.
+    ThemeProvider* provider = widget->GetThemeProvider();
+    if (provider)
+      return provider;
+
+    provider = widget->GetDefaultThemeProvider();
     if (provider)
       return provider;
   }