linux_aura: Default to system titlebars under some window managers.

chrome/browser/ui/gtk/browser_window_gtk.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/gtk/browser_window_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 
 #include <algorithm>
 #include <string>
@@ skipping 1510 matching lines @@
   return titlebar_->widget();
 }
 
 // static
 void BrowserWindowGtk::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   bool custom_frame_default = false;
   // Avoid checking the window manager if we're not connected to an X server (as
   // is the case in Valgrind tests).
   if (ui::XDisplayExists())
-    custom_frame_default = GetCustomFramePrefDefault();
+    custom_frame_default = ui::GetCustomFramePrefDefault();
 
   registry->RegisterBooleanPref(
       prefs::kUseCustomChromeFrame,
       custom_frame_default,
       user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
 }
 
 WebContents* BrowserWindowGtk::GetDisplayedTab() {
   return contents_container_->tab();
 }
@@ skipping 806 matching lines @@
 
 void BrowserWindowGtk::OnUseCustomChromeFrameChanged() {
   UpdateCustomFrame();
   ui::SetHideTitlebarWhenMaximizedProperty(
       ui::GetX11WindowFromGtkWidget(GTK_WIDGET(window_)),
       UseCustomFrame() ? ui::HIDE_TITLEBAR_WHEN_MAXIMIZED :
                          ui::SHOW_TITLEBAR_WHEN_MAXIMIZED);
 }
 
 // static
-bool BrowserWindowGtk::GetCustomFramePrefDefault() {
-  // Ideally, we'd use the custom frame by default and just fall back on using
-  // system decorations for the few (?) tiling window managers where the custom
-  // frame doesn't make sense (e.g. awesome, ion3, ratpoison, xmonad, etc.) or
-  // other WMs where it has issues (e.g. Fluxbox -- see issue 19130).  The EWMH
-  // _NET_SUPPORTING_WM property makes it easy to look up a name for the current
-  // WM, but at least some of the WMs in the latter group don't set it.
-  // Instead, we default to using system decorations for all WMs and
-  // special-case the ones where the custom frame should be used.
-  ui::WindowManagerName wm_type = ui::GuessWindowManager();
-  return (wm_type == ui::WM_BLACKBOX ||
-          wm_type == ui::WM_COMPIZ ||
-          wm_type == ui::WM_ENLIGHTENMENT ||
-          wm_type == ui::WM_METACITY ||
-          wm_type == ui::WM_MUFFIN ||
-          wm_type == ui::WM_MUTTER ||
-          wm_type == ui::WM_OPENBOX ||
-          wm_type == ui::WM_XFWM4);
-}
-
-// static
 BrowserWindow* BrowserWindow::CreateBrowserWindow(Browser* browser) {
   BrowserWindowGtk* browser_window_gtk = new BrowserWindowGtk(browser);
   browser_window_gtk->Init();
   return browser_window_gtk;
 }
 
 // static
 chrome::HostDesktopType BrowserWindow::AdjustHostDesktopType(
     chrome::HostDesktopType desktop_type) {
   return desktop_type;
 }