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

Unified Diff: chrome/browser/ui/gtk/browser_titlebar.cc

Issue 9956145: Remove wrench button from panel. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix per feedback Created 8 years, 8 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
« no previous file with comments | « chrome/browser/ui/gtk/browser_titlebar.h ('k') | chrome/browser/ui/gtk/browser_window_gtk.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/browser_titlebar.cc
diff --git a/chrome/browser/ui/gtk/browser_titlebar.cc b/chrome/browser/ui/gtk/browser_titlebar.cc
index a1b3ed2465c91359b6b18dfef1356ecdf6122aaa..d030bcf7ecf79300721ef78d1b60996f5b44a241 100644
--- a/chrome/browser/ui/gtk/browser_titlebar.cc
+++ b/chrome/browser/ui/gtk/browser_titlebar.cc
@@ -383,16 +383,6 @@ void BrowserTitlebar::Init() {
reinterpret_cast<void*>(true));
gtk_container_add(GTK_CONTAINER(favicon_event_box), app_mode_favicon_);
- if (IsTypePanel()) {
- panel_wrench_button_.reset(
- BuildTitlebarButton(IDR_BALLOON_WRENCH, IDR_BALLOON_WRENCH_P,
- IDR_BALLOON_WRENCH_H, app_mode_hbox, FALSE,
- IDS_PANEL_WINDOW_SETTINGS_BUTTON_TOOLTIP));
- g_signal_connect(panel_wrench_button_->widget(), "button-press-event",
- G_CALLBACK(OnPanelSettingsMenuButtonPressedThunk), this);
- gtk_widget_set_no_show_all(panel_wrench_button_->widget(), TRUE);
- }
-
app_mode_title_ = gtk_label_new(NULL);
gtk_label_set_ellipsize(GTK_LABEL(app_mode_title_), PANGO_ELLIPSIZE_END);
gtk_misc_set_alignment(GTK_MISC(app_mode_title_), 0.0, 0.5);
@@ -925,15 +915,6 @@ gboolean BrowserTitlebar::OnFaviconMenuButtonPressed(GtkWidget* widget,
return TRUE;
}
-gboolean BrowserTitlebar::OnPanelSettingsMenuButtonPressed(
- GtkWidget* widget, GdkEventButton* event) {
- if (event->button != 1)
- return FALSE;
-
- browser_window_->ShowSettingsMenu(widget, event);
- return TRUE;
-}
-
void BrowserTitlebar::ShowContextMenu(GdkEventButton* event) {
if (!context_menu_.get()) {
context_menu_model_.reset(new ContextMenuModel(this));
@@ -988,16 +969,6 @@ int BrowserTitlebar::IconOnlyWidth() {
return 2 * kFrameBorderThickness + allocation.width;
}
-void BrowserTitlebar::ShowPanelWrenchButton() {
- if (panel_wrench_button_.get())
- gtk_widget_show(panel_wrench_button_->widget());
-}
-
-void BrowserTitlebar::HidePanelWrenchButton() {
- if (panel_wrench_button_.get())
- gtk_widget_hide(panel_wrench_button_->widget());
-}
-
bool BrowserTitlebar::IsCommandIdEnabled(int command_id) const {
if (command_id == kShowWindowDecorationsCommand)
return true;
« no previous file with comments | « chrome/browser/ui/gtk/browser_titlebar.h ('k') | chrome/browser/ui/gtk/browser_window_gtk.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698