Update permission bubble anchor when omnibar is hidden

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

Index: chrome/browser/ui/views/frame/browser_view.h
diff --git a/chrome/browser/ui/views/frame/browser_view.h b/chrome/browser/ui/views/frame/browser_view.h
index a85e479bd12b18442391c6724220b1c8e7c614dd..8930b372cc50fa3ea8ab525e8c874c59fb5ed2de 100644
--- a/chrome/browser/ui/views/frame/browser_view.h
+++ b/chrome/browser/ui/views/frame/browser_view.h
@@ -125,6 +125,11 @@ class BrowserView : public BrowserWindow,
   // bubbles it creates will be associated with a single visible tab.
   void InitPermissionBubbleView();
 
+  // Updates the permission bubble view. It's necessary to update the bubble
+  // view so that it is still relevant when the window transitions to/from
+  // fullscreen.

[groby-ooo-7-16, 2015/02/28 00:25:12]

Technically, no. The bubble gets dismissed on a window transition, no?

[hcarmona, 2015/02/28 01:14:16]

The permission bubble remains visible even on a window transition. This comment
is not clear enough that it will only update for the next time a new bubble is
shown.

Updated the comment, but you bring up a good point: should the permission
bubbles be dismissed?

[felt, 2015/03/03 01:04:35]

Good catch, I hadn't tested this.

It should remain visible, but moved into the center (like what your current CL
is doing otherwise if the omnibox is not visible).

The tricky thing here is that there also will be a full screen ephemeral prompt
in the same / nearby place, and it doesn't make sense to put them on top of each
other.

BTW you can test here: https://adrifelt.github.io/demos/all-permissions.html it
has both notifications (for prompt) and full screen mode.

[hcarmona, 2015/03/06 23:24:16]

Where should we move the promt to? Centering the prompt does make it look very
weird because it overlaps the full screen prompt.

[felt, 2015/03/07 03:31:55]

Is it possible to tell if the full screen prompt is also displayed, and if so,
push it down below it?

+  void UpdatePermissionBubbleView();
+
   // Returns the apparent bounds of the toolbar, in BrowserView coordinates.
   // These differ from |toolbar_.bounds()| in that they match where the toolbar
   // background image is drawn -- slightly outside the "true" bounds