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

Unified Diff: ui/views/widget/desktop_aura/desktop_native_widget_aura.cc

Issue 136093007: Widget::ShouldUseNativeFrame is now meaningful on Linux. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 6 years, 11 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: ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
diff --git a/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc b/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
index dd6a3cf0f80f40648098ab52b72c11c80d00796e..9aab0cb99aee721e7c9c5208857623a3c8655661 100644
--- a/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
+++ b/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
@@ -1157,7 +1157,13 @@ void DesktopNativeWidgetAura::InstallInputMethodEventFilter() {
}
void DesktopNativeWidgetAura::UpdateWindowTransparency() {
- content_window_->SetTransparent(ShouldUseNativeFrame());
+ bool transparent = false;
+#if defined(OS_WIN)
+ // On Windows, a native frame implies Glass, so the window should be
+ // transparent.
+ transparent = ShouldUseNativeFrame();
sky 2014/02/03 21:53:03 Add a method to DesktopRootWindow that determines
Matt Giuca 2014/02/05 23:25:31 Done.
sky 2014/02/06 00:36:14 This isn't quite what I was asking for. I'm saying
Matt Giuca 2014/02/07 09:13:11 Done.
+#endif
+ content_window_->SetTransparent(transparent);
}
} // namespace views

Powered by Google App Engine
This is Rietveld 408576698