Update permission bubble anchor when omnibar is hidden

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

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 109 matching lines @@
 
   // Initializes (or re-initializes) the status bubble.  We try to only create
   // the bubble once and re-use it for the life of the browser, but certain
   // events (such as changing enabling/disabling Aero on Win) can force a need
   // to change some of the bubble's creation parameters.
   void InitStatusBubble();
 
   // Initializes the permission bubble view. This class is intended to be
   // created once and then re-used for the life of the browser window. The
   // bubbles it creates will be associated with a single visible tab.
   void InitPermissionBubbleView();

[msw, 2015/03/12 20:44:05]

We should probably remove InitPermissionBubbleView altogether, this isn't
browser frame UI that needs to be initialized with the browser frame, it's
transient UI that should be created when/if needed (and destroyed on close).

[hcarmona, 2015/03/17 01:29:27]

Done.

 
+  // It's necessary to update the bubble view so that it is in the correct
+  // location the next time it needs to be shown.
+  void UpdatePermissionBubbleViewAnchor();
+
   // 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
   // horizontally. Note that this returns the bounds for the toolbar area.
   gfx::Rect GetToolbarBounds() const;
 
   // Returns the constraining bounding box that should be used to lay out the
   // FindBar within. This is _not_ the size of the find bar, just the bounding
   // box it should be laid out within. The coordinate system of the returned
   // rect is in the coordinate system of the frame, since the FindBar is a child
@@ skipping 573 matching lines @@
   scoped_ptr<ImmersiveModeController> immersive_mode_controller_;
 
   scoped_ptr<WebContentsCloseHandler> web_contents_close_handler_;
 
   mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_