Mac: Add a shortcut to open the Apps page from the bookmark bar.

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm

Index: chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm
index 92b60df3f20e200110eae0447c0b75ca6e3d7822..32be4b64151620aa35389286246646ac1c000a77 100644
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.mm
@@ -4,10 +4,16 @@
 
 #include "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.h"
 
+#include "base/prefs/pref_service.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.h"
+#include "chrome/common/pref_names.h"
 
-BookmarkBarBridge::BookmarkBarBridge(BookmarkBarController* controller,
+
+BookmarkBarBridge::BookmarkBarBridge(Browser* browser,
+                                     BookmarkBarController* controller,
                                      BookmarkModel* model)
     : controller_(controller),
       model_(model),
@@ -18,6 +24,12 @@ BookmarkBarBridge::BookmarkBarBridge(BookmarkBarController* controller,
   // We will be notified when that happens with the AddObserver() call.
   if (model->IsLoaded())
     Loaded(model, false);
+
+  profile_pref_registrar_.Init(browser->profile()->GetPrefs());

[Alexei Svitkine (slow), 2013/03/07 21:17:07]

Are you only using |browser| to get a profile? If so, pass in a profile.

[John Grabowski, 2013/03/07 21:35:14]

+1

[beaudoin, 2013/03/08 00:02:50]

Done.

+  profile_pref_registrar_.Add(
+      prefs::kShowAppsShortcutInBookmarkBar,
+      base::Bind(&BookmarkBarBridge::OnAppsPageShortcutVisibilityChanged,
+      base::Unretained(this)));

[Alexei Svitkine (slow), 2013/03/07 21:17:07]

Indent this line 4 more.

[beaudoin, 2013/03/08 00:02:50]

Done.

 }
 
 BookmarkBarBridge::~BookmarkBarBridge() {
@@ -81,3 +93,7 @@ void BookmarkBarBridge::ExtensiveBookmarkChangesEnded(BookmarkModel* model) {
   batch_mode_ = false;
   [controller_ loaded:model];
 }
+
+void BookmarkBarBridge::OnAppsPageShortcutVisibilityChanged() {
+  [controller_ updateAppsPageShortcutButtonVisibility];
+}