[MacViews] Anchor Bookmarks bubble to top-right corner.

chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc

Index: chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc b/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
index e3e5e0aabaf991eaf66d8ec428addf06c11f37f2..00aff6bf2f580b6efbe5ae0ea465a391be6fb15b 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
@@ -80,6 +80,10 @@ views::Widget* BookmarkBubbleView::ShowBubble(
   bookmark_bubble_ =
       new BookmarkBubbleView(anchor_view, observer, std::move(delegate),
                              profile, url, !already_bookmarked);
+  // Bookmark bubble should always anchor TOP_RIGHT, but the

[sky, 2016/09/20 02:50:21]

If the bookmark bubble should always anchor like you say, should this code move
to the BookmarkBubbleView constructor?

[Eugene But (OOO till 7-30), 2016/09/20 03:11:23]

That would feel inconsistent with the code below (inside !anchor_view block). Do
you think I should move it to constructor anyways?

+  // LocationBarBubbleDelegateView does not know that and may use different
+  // arrow anchoring.
+  bookmark_bubble_->set_arrow(views::BubbleBorder::TOP_RIGHT);
   if (!anchor_view) {
     bookmark_bubble_->SetAnchorRect(anchor_rect);
     bookmark_bubble_->set_parent_window(parent_window);