Ensure that when themes in desktop chrome AURA on Windows use the opaque browser frame view.

chrome/browser/ui/views/frame/desktop_browser_frame_aura.cc

Index: chrome/browser/ui/views/frame/desktop_browser_frame_aura.cc
===================================================================
--- chrome/browser/ui/views/frame/desktop_browser_frame_aura.cc	(revision 181966)
+++ chrome/browser/ui/views/frame/desktop_browser_frame_aura.cc	(working copy)
@@ -19,7 +19,12 @@
 #include "ui/gfx/font.h"
 #include "ui/views/corewm/visibility_controller.h"
 #include "ui/views/view.h"
+#include "ui/views/widget/widget.h"
 
+#if defined(OS_WIN)
+#include "chrome/browser/ui/views/frame/browser_frame_common_win.h"
+#endif
+
 using aura::Window;
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -86,6 +91,17 @@
 void DesktopBrowserFrameAura::TabStripDisplayModeChanged() {
 }
 
+bool DesktopBrowserFrameAura::ShouldUseNativeFrame() const {
+#if !defined(OS_WIN)

[sky, 2013/02/13 00:42:54]

Can you forward this to browser_desktop_root_window_host_ so we don't need the
ifdefs here?

[ananta, 2013/02/13 01:43:21]

I removed this method which achieves what you described. The 
DesktopNativeWidgetAura::ShouldUseNativeFrame override already does this, i.e.
forwards it off to the desktop_root_window_host_. I added a windows specific
implementation of this function in the BrowserDesktopRootWindowHostWin which
forwards it off to the chrome::ShouldUseNativeFrameHelper.

PTAL.

+  return views::DesktopNativeWidgetAura::ShouldUseNativeFrame();
+#else
+  return browser::ShouldUseNativeFrame(this,
+                                       browser_view_,
+                                       GetWidget()->GetThemeProvider());
+#endif
+}
+
+
 ///////////////////////////////////////////////////////////////////////////////
 // DesktopBrowserFrameAura, private: