Introduce ChromeBookmarkClientFactory

chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h

Index: chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h b/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h
index 502430f66263e3202698f2b27aff0c681fd96b50..ef43eb6bdb4d6b3760aa92af84819c7e925a8254 100644
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_bubble_controller.h
@@ -9,6 +9,7 @@
 #import "chrome/browser/ui/cocoa/base_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_model_observer_for_cocoa.h"
 
+class BookmarkModel;
 class BookmarkNode;
 class ChromeBookmarkClient;
 @class BookmarkBubbleController;
@@ -20,8 +21,10 @@ class ChromeBookmarkClient;
 // the bookmark in various ways (name, folder, etc.)
 @interface BookmarkBubbleController : BaseBubbleController {
  @private
-  // Both weak; owned by the current browser's profile.
+  // |client_|, |model_| and |node_| are weak and owned by the current browser's
+  // profile.
   ChromeBookmarkClient* client_;  // weak
+  BookmarkModel* model_;          // weak
   const BookmarkNode* node_;  // weak

[Scott Hess - ex-Googler, 2014/06/11 18:31:11]

Pick your layout - either two spaces for all three, or line all three up!

[sdefresne, 2014/06/11 21:05:46]

This was formatted with clang-format. Probably a bug of the tool. Picked two
spaces.

 
   // The bookmark node whose button we asked to pulse.
@@ -51,6 +54,7 @@ class ChromeBookmarkClient;
 // init routine.  Closing of the window happens implicitly on dealloc.
 - (id)initWithParentWindow:(NSWindow*)parentWindow
                     client:(ChromeBookmarkClient*)client
+                     model:(BookmarkModel*)model
                       node:(const BookmarkNode*)node
          alreadyBookmarked:(BOOL)alreadyBookmarked;