Anchor the first run bubble off the location bar icon's ImageView.

chrome/browser/ui/views/location_bar/location_bar_view.cc

Index: chrome/browser/ui/views/location_bar/location_bar_view.cc
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.cc b/chrome/browser/ui/views/location_bar/location_bar_view.cc
index a2cbbbe733709e5a2f77e9139b2991e360eebf81..232a2228475e0a684c3422a51b6d5685df6edc82 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.cc
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.cc
@@ -853,7 +853,7 @@ void LocationBarView::ShowFirstRunBubbleInternal() {
     return;
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
   if (browser)
-    FirstRunBubble::ShowBubble(browser, location_icon_view_);
+    FirstRunBubble::ShowBubble(browser, location_icon_view_->GetImageView());

[Peter Kasting, 2017/02/23 22:06:03]

It seems like this should just call GetSecurityBubbleAnchorView().  (If the name
is now too specific, we could rename this, although I don't have any great name
suggestions.)

Otherwise it seems like we will need to fix this again for Harmony, to just
duplicate that code.

[tapted, 2017/02/23 23:19:48]

Done.

I had debated this and couldn't decide :). We don't have mocks for the first run
bubble. The bubble says "you can search from here with Google" and points to a
magnifying glass. Seems weird to have a bubble that says "look here" and doesn't
have an arrow on it, but the way BubbleFrameView is set up under harmony, the
arrow is removed regardless of where we anchor it (and I guess it doesn't look
too bad).

 #endif
 }