[Mac] Enable MacViews bookmark bubble behind --enable-mac-views-dialogs.

chrome/browser/ui/views/browser_dialogs_views_mac.cc

Index: chrome/browser/ui/views/browser_dialogs_views_mac.cc
diff --git a/chrome/browser/ui/views/browser_dialogs_views_mac.cc b/chrome/browser/ui/views/browser_dialogs_views_mac.cc
index 61a5f8762a4355ce8eb694784fd9e11f7389fcc2..8bff2959e7be054f12147127233bfdc56f24a203 100644
--- a/chrome/browser/ui/views/browser_dialogs_views_mac.cc
+++ b/chrome/browser/ui/views/browser_dialogs_views_mac.cc
@@ -4,6 +4,9 @@
 
 #include "chrome/browser/ui/browser_dialogs.h"
 
+#include "chrome/browser/ui/bookmarks/bookmark_bubble_sign_in_delegate.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/views/bookmarks/bookmark_bubble_view.h"
 #include "chrome/browser/ui/views/website_settings/website_settings_popup_view.h"
 
 // This file provides definitions of desktop browser dialog-creation methods for
@@ -24,4 +27,19 @@ void ShowWebsiteSettingsBubbleViewsAtPoint(const gfx::Point& anchor_point,
                                       profile, web_contents, url, ssl);
 }
 
+void ShowBookmarkBubbleViewsAtPoint(const gfx::Point& anchor_point,
+                                    gfx::NativeView parent,
+                                    bookmarks::BookmarkBubbleObserver* observer,
+                                    Browser* browser,
+                                    const GURL& url,
+                                    bool already_bookmarked) {

[msw, 2015/08/26 00:00:47]

optional nit: use inverted |newly_bookmarked| to match views (or flip others).

[jackhou1, 2015/08/26 00:44:40]

Flipped others. BrowserWindow::ShowBookmarkBubble has it as |already_bookmarked|
and this CL already touches browser_view.cc. Now the inversion is confined to
bookmark_bubble_view.

+  // The Views dialog may prompt for sign in.
+  scoped_ptr<BookmarkBubbleDelegate> delegate;
+  delegate.reset(new BookmarkBubbleSignInDelegate(browser));

[msw, 2015/08/26 00:00:47]

optional nit: inline in |delegate| decl or in ShowBubble with make_scoped_ptr.

[jackhou1, 2015/08/26 00:44:40]

Done.

+
+  BookmarkBubbleView::ShowBubble(nullptr, gfx::Rect(anchor_point, gfx::Size()),
+                                 parent, observer, delegate.Pass(),
+                                 browser->profile(), url, !already_bookmarked);
+}
+
 }  // namespace chrome