Update permission bubble anchor when omnibar is hidden

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index b86f3974716cc936f81e7583c84de815b331955c..1bbb5f49e4dc1acfac3036a3c548c26fb8a76467 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -540,10 +540,26 @@ void BrowserView::InitStatusBubble() {
 }
 
 void BrowserView::InitPermissionBubbleView() {
-  std::string languages =
-      browser_->profile()->GetPrefs()->GetString(prefs::kAcceptLanguages);
   permission_bubble_view_.reset(new PermissionBubbleViewViews(
-      GetLocationBarView()->location_icon_view(), languages));
+      browser_->profile()->GetPrefs()->GetString(prefs::kAcceptLanguages)));
+  UpdatePermissionBubbleViewAnchor();
+}
+
+void BrowserView::UpdatePermissionBubbleViewAnchor() {

[msw, 2015/03/11 22:25:30]

Other bubbles (bookmark, global error, avatar/profile, etc.) don't have this
special behavior in fullscreen (no arrow, anchor to the fullscreen bubble or top
container). I think we ought to be consistent about the behavior of these
bubbles... please ping UX about the right behavior.

I'd generally support your solution here, but only if the same behavior were
recommended for the other bubbles (with bugs filed to follow up), and we had
some mechanism [planned] to handle overlapping bubbles (what if the permission
bubble is showing and the user hits CTRL+D for the bookmark bubble, CTRL+M for
the avatar/profile bubble, or gets a global error bubble, etc.)...

[groby-ooo-7-16, 2015/03/12 00:44:49]

Hector: The OSX version manages without modifications to the BrowserView. Can we
take the same approach on views?

If we can't, can the bubble view listen to a fullscreen change notification and
just re-query its anchor point, as opposed to hard-coding the dependency here?

[hcarmona, 2015/03/12 18:43:33]

Is there a specific person I should ping, or should I send a message to
chrome-ux@?
Unfortunately, no because in OSX when fullscreen toggles the bubbles still have
an anchor. This is necessary in order to re-anchor the permission bubbles. Most
bubbles are dismissed in the transition, but permission bubbles are special
because users should interact with them.
I was thinking that this would be a good place to send the notification because
BrowserView already knows about both the anchor point and the permission bubble.

[groby-ooo-7-16, 2015/03/12 19:27:46]

ainslie@ is looking at all things bubble. felt@ can tell you how important it is
to push this for permissions, even if we don't unify all the bubbles yet. (My
take is that it's important enough to justify a one-off)
I'll defer to msw@ on that - he's more familiar with the views side of things.

+  if (browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR)) {
+    permission_bubble_view_->SetAnchor(
+        GetLocationBarView()->location_icon_view(),
+        views::BubbleBorder::TOP_LEFT);
+  } else {
+    views::View* view = NULL;

[msw, 2015/03/11 22:25:30]

nit: init |view| to |top_container_| to avoid the else below.

[hcarmona, 2015/03/12 18:43:33]

Done.

+
+    if (IsFullscreenBubbleVisible())
+      view = exclusive_access_bubble_->GetView();
+    else
+      view = top_container_;
+
+    permission_bubble_view_->SetAnchor(view, views::BubbleBorder::NONE);
+  }
 }
 
 gfx::Rect BrowserView::GetToolbarBounds() const {
@@ -2291,6 +2307,8 @@ void BrowserView::ProcessFullscreen(bool fullscreen,
   // order to let the layout occur.
   in_process_fullscreen_ = false;
   ToolbarSizeChanged(false);
+
+  UpdatePermissionBubbleViewAnchor();
 }
 
 bool BrowserView::ShouldUseImmersiveFullscreenForUrl(const GURL& url) const {