Remove wrench button from panel.

chrome/browser/ui/gtk/browser_titlebar.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.
 //
 // A helper class that contains the gtk widgets that make up the titlebar.  The
 // titlebar consists of the tabstrip and if the custom chrome frame is turned
 // on, it includes the taller titlebar and minimize, restore, maximize, and
 // close buttons.
 
 #ifndef CHROME_BROWSER_UI_GTK_BROWSER_TITLEBAR_H_
@@ skipping 57 matching lines @@
   // |web_contents| is the tab that is associated with the window throbber.
   // |web_contents| can be null.
   void UpdateThrobber(content::WebContents* web_contents);
 
   // On Windows, right clicking in the titlebar background brings up the system
   // menu.  There's no such thing on linux, so we just show the menu items we
   // add to the menu.
   void ShowContextMenu(GdkEventButton* event);
 
   // When a panel slides into a new position and the cursor is on the close
-  // button, the close button does not become clickable. The gtk_widget_show()
-  // call on panel_wrench_button_ in OnEnterNotify on window_ prevents the
-  // close_button_ from getting the enter-notify-event, making it unclickable.
+  // button, the close button does not become clickable. Neither of

[Evan Stade, 2012/04/16 19:23:48]

grammar

[jianli, 2012/04/16 23:24:45]

Done.

+  // "enter-notify-event" or "clicked" event on the mouse does not fire.
   // This creates a bad experience when a user has multiple panels of the same
   // size (which is typical) and tries closing them all by repeatedly clicking
   // in the same place on the screen.
   //
   // Opened a gtk bug for this -
   //   https://bugzilla.gnome.org/show_bug.cgi?id=667841
   void SendEnterNotifyToCloseButtonIfUnderMouse();

[jennb, 2012/04/16 21:17:05]

Is this still needed now that there's no wrench button? 

[jianli, 2012/04/16 23:24:45]

Yes, it is still needed. Update comment to make it clear.

 
   // Returns the window width to display just the icon.
   int IconOnlyWidth();
 
-  void ShowPanelWrenchButton();
-  void HidePanelWrenchButton();
-
   AvatarMenuButtonGtk* avatar_button() { return avatar_button_.get(); }
 
  private:
   // A helper class to keep track of which frame of the throbber animation
   // we're showing.
   class Throbber {
    public:
     Throbber() : current_frame_(0), current_waiting_frame_(0) {}
 
     // Get the next frame in the animation. The image is owned by the throbber
@@ skipping 63 matching lines @@
   // Callback for mousewheel events.
   CHROMEGTK_CALLBACK_1(BrowserTitlebar, gboolean, OnScroll,
                        GdkEventScroll*);
 
   // Callback for min/max/close buttons.
   CHROMEGTK_CALLBACK_0(BrowserTitlebar, void, OnButtonClicked);
 
   // Callback for favicon/settings buttons.
   CHROMEGTK_CALLBACK_1(BrowserTitlebar, gboolean,
                        OnFaviconMenuButtonPressed, GdkEventButton*);
-  CHROMEGTK_CALLBACK_1(BrowserTitlebar, gboolean,
-                       OnPanelSettingsMenuButtonPressed, GdkEventButton*);
 
   // -- Context Menu -----------------------------------------------------------
 
   // SimpleMenuModel::Delegate implementation:
   virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE;
   virtual bool IsCommandIdChecked(int command_id) const OVERRIDE;
   virtual void ExecuteCommand(int command_id) OVERRIDE;
   virtual bool GetAcceleratorForCommandId(
       int command_id,
       ui::Accelerator* accelerator) OVERRIDE;
@@ skipping 52 matching lines @@
   GtkWidget* top_padding_right_;
 
   // Gtk alignment that contains the tab strip.  If the user turns off window
   // manager decorations, we draw this taller.
   GtkWidget* titlebar_alignment_;
 
   // The favicon and page title used when in app mode or popup mode.
   GtkWidget* app_mode_favicon_;
   GtkWidget* app_mode_title_;
 
-  // Wrench icon for panels. This'll only appear when a panel window has focus
-  // or mouse is in a panel window.
-  scoped_ptr<CustomDrawButton> panel_wrench_button_;
-
   // Whether we are using a custom frame.
   bool using_custom_frame_;
 
   // Whether we have focus (gtk_window_is_active() sometimes returns the wrong
   // value, so manually track the focus-in and focus-out events.)
   bool window_has_focus_;
 
   // Whether to display the avatar image on the left or right of the titlebar.
   bool display_avatar_on_left_;
 
@@ skipping 23 matching lines @@
   // The avatar button.
   scoped_ptr<AvatarMenuButtonGtk> avatar_button_;
 
   // Theme provider for building buttons.
   ThemeServiceGtk* theme_service_;
 
   content::NotificationRegistrar registrar_;
 };
 
 #endif  // CHROME_BROWSER_UI_GTK_BROWSER_TITLEBAR_H_