Makes vertical alignment of location bar bubbles same

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 c9e0292c21a0376aae8330a8c73ca0bfc92b37df..e6166dcaa78a6f3cf0094e0136aa62d6aea0bab7 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bubble_view.cc
@@ -84,6 +84,10 @@ views::Widget* BookmarkBubbleView::ShowBubble(
     bookmark_bubble_->SetAnchorRect(anchor_rect);
     bookmark_bubble_->set_parent_window(parent_window);
   }
+  // Compensate for built-in vertical padding in the anchor view's image.
+  // This is different from 5 pixel inset that is used in other cases because
+  // this bubble is using NONE as the arrow type.
+  bookmark_bubble_->set_anchor_view_insets(gfx::Insets(-1, 0));

[Peter Kasting, 2016/03/14 22:35:51]

Do you still need to do this if you use PAINT_TRANSPARENT for the arrow type
instead?  I think that should position the bubble like the others but just not
draw an arrow.

[varkha, 2016/03/15 00:11:42]

That would indeed be great and I wanted to do that. However with the arrow code
as it is the transparent arrow is not transparent to clicks so it will be a
noticeable regression in how easy it is to click the star when a bubble is open
(assuming the same visible vertical offset).
I will look into whether it is possible to update hit testing for the bubble to
account for a transparent arrow but prefer to do that separately.

[Evan Stade, 2016/03/15 00:13:27]

I think it should be possible and it's probably a decent idea to look into that
before landing this patch since it has the potential to affect this patch.

[Peter Kasting, 2016/03/15 00:34:15]

Yeah, I'd like to see if this is easy enough to do as well.  Doing it separately
is fine but I would go ahead and look at that first.

[varkha, 2016/03/16 03:45:30]

I have tried to pursue this. I can make the BubbleFrameView transparent to
events by overriding DoesIntersectRect (similar to [1]) but that doesn't solve
the problem because the bubble has its own native widget and so the mouse events
never reach the browser frame's root view. Am I missing something easy?
[1] -
https://codereview.chromium.org/1767363002/diff/80001/chrome/browser/ui/views...

   views::Widget* bubble_widget =
       views::BubbleDialogDelegateView::CreateBubble(bookmark_bubble_);
   bubble_widget->Show();
@@ -273,8 +277,6 @@ BookmarkBubbleView::BookmarkBubbleView(
       parent_combobox_(NULL),
       remove_bookmark_(false),
       apply_edits_(true) {
-  // Compensate for built-in vertical padding in the anchor view's image.
-  set_anchor_view_insets(gfx::Insets(2, 0, 2, 0));
 }
 
 base::string16 BookmarkBubbleView::GetTitle() {