diff options
author | khorimoto@chromium.org <khorimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-09 21:58:54 +0000 |
---|---|---|
committer | khorimoto@chromium.org <khorimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-09 21:58:54 +0000 |
commit | ded4bf0a33465317cc483db4856f797555ca5311 (patch) | |
tree | 6ecc621a6844fbcd5d08ea9bc49b6978e2d39149 /chrome | |
parent | 9f6c727335d997a1f7def72531239abceb299e60 (diff) | |
download | chromium_src-ded4bf0a33465317cc483db4856f797555ca5311.zip chromium_src-ded4bf0a33465317cc483db4856f797555ca5311.tar.gz chromium_src-ded4bf0a33465317cc483db4856f797555ca5311.tar.bz2 |
Moved code to create zoom button outside of an if statement. The current location would have prohibited creating the icon if bookmarks were disabled.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10703084
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145753 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/ui/gtk/location_bar_view_gtk.cc | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/chrome/browser/ui/gtk/location_bar_view_gtk.cc b/chrome/browser/ui/gtk/location_bar_view_gtk.cc index 94a4bbf..30fd72d 100644 --- a/chrome/browser/ui/gtk/location_bar_view_gtk.cc +++ b/chrome/browser/ui/gtk/location_bar_view_gtk.cc @@ -324,17 +324,11 @@ void LocationBarViewGtk::Init(bool popup_window_mode) { // doesn't work, someone is probably calling show_all on our parent box. gtk_box_pack_end(GTK_BOX(entry_box_), tab_to_search_hint_, FALSE, FALSE, 0); - // Hide the zoom, star and Chrome To Mobile icons in popups, app windows, etc. + // Hide the star and Chrome To Mobile icons in popups, app windows, etc. if (browser_defaults::bookmarks_enabled && !ShouldOnlyShowLocation()) { CreateStarButton(); gtk_box_pack_end(GTK_BOX(hbox_.get()), star_.get(), FALSE, FALSE, 0); - // Zoom code is commented out for now because it is not complete for all - // platforms, so it should not be present in M21. - // TODO(khorimoto): Uncomment the code below once it is ready. - // CreateZoomButton(); - // gtk_box_pack_end(GTK_BOX(hbox_.get()), zoom_.get(), FALSE, FALSE, 0); - // Also disable Chrome To Mobile for off-the-record and non-synced profiles, // or if the feature is disabled by a command line flag or chrome://flags. if (!profile->IsOffTheRecord() && profile->IsSyncAccessible() && @@ -348,6 +342,12 @@ void LocationBarViewGtk::Init(bool popup_window_mode) { } } + // Zoom code is commented out for now because it is not complete for all + // platforms, so it should not be present in M21. + // TODO(khorimoto): Uncomment the code below once it is ready. + // CreateZoomButton(); + // gtk_box_pack_end(GTK_BOX(hbox_.get()), zoom_.get(), FALSE, FALSE, 0); + content_setting_hbox_.Own(gtk_hbox_new(FALSE, kInnerPadding + 1)); gtk_widget_set_name(content_setting_hbox_.get(), "chrome-content-setting-hbox"); |