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

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

Issue 12377065: Consolidate VIEW_ID_LOCATION_BAR into VIEW_ID_OMNIBOX. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/ui/gtk/browser_toolbar_gtk.h" 5 #include "chrome/browser/ui/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 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 SetViewIDs(); 273 SetViewIDs();
274 theme_service_->InitThemesFor(this); 274 theme_service_->InitThemesFor(this);
275 } 275 }
276 276
277 void BrowserToolbarGtk::SetViewIDs() { 277 void BrowserToolbarGtk::SetViewIDs() {
278 ViewIDUtil::SetID(widget(), VIEW_ID_TOOLBAR); 278 ViewIDUtil::SetID(widget(), VIEW_ID_TOOLBAR);
279 ViewIDUtil::SetID(back_->widget(), VIEW_ID_BACK_BUTTON); 279 ViewIDUtil::SetID(back_->widget(), VIEW_ID_BACK_BUTTON);
280 ViewIDUtil::SetID(forward_->widget(), VIEW_ID_FORWARD_BUTTON); 280 ViewIDUtil::SetID(forward_->widget(), VIEW_ID_FORWARD_BUTTON);
281 ViewIDUtil::SetID(reload_->widget(), VIEW_ID_RELOAD_BUTTON); 281 ViewIDUtil::SetID(reload_->widget(), VIEW_ID_RELOAD_BUTTON);
282 ViewIDUtil::SetID(home_->widget(), VIEW_ID_HOME_BUTTON); 282 ViewIDUtil::SetID(home_->widget(), VIEW_ID_HOME_BUTTON);
283 ViewIDUtil::SetID(location_bar_->widget(), VIEW_ID_LOCATION_BAR); 283 ViewIDUtil::SetID(location_bar_->widget(), VIEW_ID_OMNIBOX);
284 ViewIDUtil::SetID(wrench_menu_button_->widget(), VIEW_ID_APP_MENU); 284 ViewIDUtil::SetID(wrench_menu_button_->widget(), VIEW_ID_APP_MENU);
285 } 285 }
286 286
287 void BrowserToolbarGtk::Show() { 287 void BrowserToolbarGtk::Show() {
288 gtk_widget_show(toolbar_); 288 gtk_widget_show(toolbar_);
289 } 289 }
290 290
291 void BrowserToolbarGtk::Hide() { 291 void BrowserToolbarGtk::Hide() {
292 gtk_widget_hide(toolbar_); 292 gtk_widget_hide(toolbar_);
293 } 293 }
(...skipping 410 matching lines...) Expand 10 before | Expand all | Expand 10 after
704 allocation.x + x_offset, 704 allocation.x + x_offset,
705 allocation.y + y_offset); 705 allocation.y + y_offset);
706 706
707 return FALSE; 707 return FALSE;
708 } 708 }
709 709
710 void BrowserToolbarGtk::UpdateShowHomeButton() { 710 void BrowserToolbarGtk::UpdateShowHomeButton() {
711 bool visible = show_home_button_.GetValue() && !ShouldOnlyShowLocation(); 711 bool visible = show_home_button_.GetValue() && !ShouldOnlyShowLocation();
712 gtk_widget_set_visible(home_->widget(), visible); 712 gtk_widget_set_visible(home_->widget(), visible);
713 } 713 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/view_id_util_browsertest.mm ('k') | chrome/browser/ui/gtk/view_id_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698