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

Unified Diff: chrome/browser/ui/views/bookmarks/bookmark_bar_view.h

Issue 12550006: Mac: Add a shortcut to open the Apps page from the bookmark bar. (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/bookmarks/bookmark_bar_view.h
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_bar_view.h b/chrome/browser/ui/views/bookmarks/bookmark_bar_view.h
index 7faa57919bb459e6d0611e093507d90f28aa0913..52ceb5a59d9f5ac44e832093edba96efb54ad118 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_bar_view.h
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bar_view.h
@@ -357,6 +357,10 @@ class BookmarkBarView : public DetachableToolbarView,
// the separator if required.
void UpdateOtherBookmarksVisibility();
+ // Updates the visibility of |apps_page_shortcut_|. Also shows or hide the
+ // separator if required.
+ void UpdateAppsPageShortcutVisibility();
+
// Updates the visibility of |bookmarks_separator_view_|.
void UpdateBookmarksSeparatorVisibility();
@@ -366,9 +370,6 @@ class BookmarkBarView : public DetachableToolbarView,
// desired bounds. If |compute_bounds_only| = FALSE, the bounds are set.
gfx::Size LayoutItems(bool compute_bounds_only);
- // Returns true if we should show the apps shortcut.
- bool ShouldShowAppsShortcut() const;
-
// Updates the visibility of the apps shortcut based on the pref value.
void OnAppsPageShortcutVisibilityChanged();

Powered by Google App Engine
This is Rietveld 408576698