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

Side by Side Diff: chrome/browser/gtk/browser_toolbar_gtk.cc

Issue 3011007: Nukes the nano tab style on gtk and brings back the toolbar for apps (Closed)
Patch Set: Created 10 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/gtk/browser_toolbar_gtk.h" 5 #include "chrome/browser/gtk/browser_toolbar_gtk.h"
6 6
7 #include <gdk/gdkkeysyms.h> 7 #include <gdk/gdkkeysyms.h>
8 #include <gtk/gtk.h> 8 #include <gtk/gtk.h>
9 #include <X11/XF86keysym.h> 9 #include <X11/XF86keysym.h>
10 10
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 location_bar_(new LocationBarViewGtk(browser)), 88 location_bar_(new LocationBarViewGtk(browser)),
89 model_(browser->toolbar_model()), 89 model_(browser->toolbar_model()),
90 page_menu_model_(this, browser), 90 page_menu_model_(this, browser),
91 app_menu_model_(this, browser), 91 app_menu_model_(this, browser),
92 wrench_menu_model_(this, browser), 92 wrench_menu_model_(this, browser),
93 browser_(browser), 93 browser_(browser),
94 window_(window), 94 window_(window),
95 profile_(NULL), 95 profile_(NULL),
96 menu_bar_helper_(this), 96 menu_bar_helper_(this),
97 upgrade_reminder_animation_(this), 97 upgrade_reminder_animation_(this),
98 upgrade_reminder_canceled_(false), 98 upgrade_reminder_canceled_(false) {
99 collapsed_(false) {
100 browser_->command_updater()->AddCommandObserver(IDC_BACK, this); 99 browser_->command_updater()->AddCommandObserver(IDC_BACK, this);
101 browser_->command_updater()->AddCommandObserver(IDC_FORWARD, this); 100 browser_->command_updater()->AddCommandObserver(IDC_FORWARD, this);
102 browser_->command_updater()->AddCommandObserver(IDC_HOME, this); 101 browser_->command_updater()->AddCommandObserver(IDC_HOME, this);
103 browser_->command_updater()->AddCommandObserver(IDC_BOOKMARK_PAGE, this); 102 browser_->command_updater()->AddCommandObserver(IDC_BOOKMARK_PAGE, this);
104 103
105 registrar_.Add(this, 104 registrar_.Add(this,
106 NotificationType::BROWSER_THEME_CHANGED, 105 NotificationType::BROWSER_THEME_CHANGED,
107 NotificationService::AllSources()); 106 NotificationService::AllSources());
108 registrar_.Add(this, 107 registrar_.Add(this,
109 NotificationType::UPGRADE_RECOMMENDED, 108 NotificationType::UPGRADE_RECOMMENDED,
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 } 285 }
287 286
288 void BrowserToolbarGtk::Hide() { 287 void BrowserToolbarGtk::Hide() {
289 gtk_widget_hide(toolbar_); 288 gtk_widget_hide(toolbar_);
290 } 289 }
291 290
292 LocationBar* BrowserToolbarGtk::GetLocationBar() const { 291 LocationBar* BrowserToolbarGtk::GetLocationBar() const {
293 return location_bar_.get(); 292 return location_bar_.get();
294 } 293 }
295 294
296 void BrowserToolbarGtk::set_collapsed(bool val) {
297 collapsed_ = val;
298 if (collapsed_)
299 Hide();
300 else
301 Show();
302 }
303
304 void BrowserToolbarGtk::UpdateForBookmarkBarVisibility( 295 void BrowserToolbarGtk::UpdateForBookmarkBarVisibility(
305 bool show_bottom_padding) { 296 bool show_bottom_padding) {
306 gtk_alignment_set_padding(GTK_ALIGNMENT(alignment_), 297 gtk_alignment_set_padding(GTK_ALIGNMENT(alignment_),
307 ShouldOnlyShowLocation() ? 0 : kTopPadding, 298 ShouldOnlyShowLocation() ? 0 : kTopPadding,
308 !show_bottom_padding || ShouldOnlyShowLocation() ? 0 : kTopPadding, 299 !show_bottom_padding || ShouldOnlyShowLocation() ? 0 : kTopPadding,
309 0, 0); 300 0, 0);
310 } 301 }
311 302
312 void BrowserToolbarGtk::ShowPageMenu() { 303 void BrowserToolbarGtk::ShowPageMenu() {
313 if (page_menu_button_.get()) 304 if (page_menu_button_.get())
(...skipping 527 matching lines...) Expand 10 before | Expand all | Expand 10 after
841 832
842 return FALSE; 833 return FALSE;
843 } 834 }
844 835
845 bool BrowserToolbarGtk::UpgradeAnimationIsFaded() { 836 bool BrowserToolbarGtk::UpgradeAnimationIsFaded() {
846 return upgrade_reminder_animation_.cycles_remaining() > 0 && 837 return upgrade_reminder_animation_.cycles_remaining() > 0 &&
847 // This funky looking math makes the badge throb for 2 seconds once 838 // This funky looking math makes the badge throb for 2 seconds once
848 // every 8 seconds. 839 // every 8 seconds.
849 ((upgrade_reminder_animation_.cycles_remaining() - 1) / 2) % 4 == 0; 840 ((upgrade_reminder_animation_.cycles_remaining() - 1) / 2) % 4 == 0;
850 } 841 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698