Linux Aura: Use system title bar is now set by preference, not flag.

chrome/browser/ui/views/frame/browser_frame.h

 // Copyright (c) 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
+#include "base/prefs/pref_member.h"
 #include "build/build_config.h"
 #include "chrome/browser/ui/views/frame/browser_non_client_frame_view.h"
 #include "ui/views/context_menu_controller.h"
 #include "ui/views/widget/widget.h"
 
 class AvatarMenuButton;
 class BrowserRootView;
 class BrowserView;
 class NativeBrowserFrame;
 class NewAvatarButton;
@@ skipping 47 matching lines @@
 
   // Returns the amount that the theme background should be inset.
   int GetThemeBackgroundXInset() const;
 
   // Tells the frame to update the throbber.
   void UpdateThrobber(bool running);
 
   // Returns the NonClientFrameView of this frame.
   views::View* GetFrameView() const;
 
+  // Returns |true| if we should use the custom frame.
+  bool UseCustomFrame() const;
+
   // Overridden from views::Widget:
   virtual views::internal::RootView* CreateRootView() OVERRIDE;
   virtual views::NonClientFrameView* CreateNonClientFrameView() OVERRIDE;
   virtual bool GetAccelerator(int command_id,
                               ui::Accelerator* accelerator) OVERRIDE;
   virtual ui::ThemeProvider* GetThemeProvider() const OVERRIDE;
   virtual void SchedulePaintInRect(const gfx::Rect& rect) OVERRIDE;
   virtual void OnNativeWidgetActivationChanged(bool active) OVERRIDE;
 
   // Overridden from views::ContextMenuController:
   virtual void ShowContextMenuForView(views::View* source,
                                       const gfx::Point& p,
                                       ui::MenuSourceType source_type) OVERRIDE;
 
   // Returns true if we should leave any offset at the frame caption. Typically
   // when the frame is maximized/full screen we want to leave no offset at the
   // top.
   bool ShouldLeaveOffsetNearTopBorder();
 
   AvatarMenuButton* GetAvatarMenuButton();
 
   NewAvatarButton* GetNewAvatarMenuButton();
 
   // Returns the menu model. BrowserFrame owns the returned model.
   // Note that in multi user mode this will upon each call create a new model.
   ui::MenuModel* GetSystemMenuModel();
 
  private:
+  // Called when the preference changes.
+  void OnUseCustomChromeFrameChanged();
+
   NativeBrowserFrame* native_browser_frame_;
 
   // A weak reference to the root view associated with the window. We save a
   // copy as a BrowserRootView to avoid evil casting later, when we need to call
   // functions that only exist on BrowserRootView (versus RootView).
   BrowserRootView* root_view_;
 
   // A pointer to our NonClientFrameView as a BrowserNonClientFrameView.
   BrowserNonClientFrameView* browser_frame_view_;
 
   // The BrowserView is our ClientView. This is a pointer to it.
   BrowserView* browser_view_;
 
   scoped_ptr<SystemMenuModelBuilder> menu_model_builder_;
 
   // Used to show the system menu. Only used if
   // NativeBrowserFrame::UsesNativeSystemMenu() returns false.
   scoped_ptr<views::MenuRunner> menu_runner_;
 
   // SetThemeProvider() triggers setting both |owned_theme_provider_| and
   // |theme_provider_|. Initially |theme_provider_| is set to the ThemeService
   // and |owned_theme_provider_| is NULL (as ThemeServices lifetime is managed
   // externally).
   scoped_ptr<ui::ThemeProvider> owned_theme_provider_;
   ui::ThemeProvider* theme_provider_;
 
+  // Whether the custom chrome frame pref is set.  Normally you want to use

[msw, 2014/02/06 00:50:27]

nit: capitalize "Chrome"; remove "Normally you want to"; consider: "Check
UseCustomFrame()"

[Matt Giuca, 2014/02/11 04:57:43]

Done. Just removed the second sentence -- there is no point having every field
mention its associated getter. (This was just copy+pasted from
browser_window_gtk.h).

+  // UseCustomFrame() above to determine whether to use the custom frame or
+  // not.
+  BooleanPrefMember use_custom_frame_pref_;
+
   DISALLOW_COPY_AND_ASSIGN(BrowserFrame);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_