Make sure bubbles in Views default to close before their RenderFrameHosts.

components/bubble/bubble_manager.h

 // Copyright 2015 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 COMPONENTS_BUBBLE_BUBBLE_MANAGER_H_
 #define COMPONENTS_BUBBLE_BUBBLE_MANAGER_H_
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
@@ skipping 29 matching lines @@
    private:
     DISALLOW_COPY_AND_ASSIGN(BubbleManagerObserver);
   };
 
   // Should be instantiated on the UI thread.
   BubbleManager();
   virtual ~BubbleManager();
 
   // Shows a specific bubble and returns a reference to it.
   // This reference should be used through the BubbleManager.
+  // Unless the BubbleDelegate uses ShouldClose to override the default, the
+  // BubbleDelegate will be destroyed before any RenderFrameHost is destroyed.

[hcarmona, 2016/01/11 19:07:22]

Same concern as above about this being very specific to the
RenderFrameHost. Can this comment go with the enum?

   BubbleReference ShowBubble(scoped_ptr<BubbleDelegate> bubble);
 
   // Notify a bubble of an event that might trigger close.
   // Returns true if the bubble was actually closed.
   bool CloseBubble(BubbleReference bubble, BubbleCloseReason reason);
 
   // Notify all bubbles of an event that might trigger close.
   void CloseAllBubbles(BubbleCloseReason reason);
 
   // Notify all bubbles that their anchor or parent may have changed.
@@ skipping 30 matching lines @@
   // Determines what happens to a bubble when |ShowBubble| is called.
   ManagerState manager_state_;
 
   // The bubbles that are being managed.
   ScopedVector<BubbleController> controllers_;
 
   DISALLOW_COPY_AND_ASSIGN(BubbleManager);
 };
 
 #endif  // COMPONENTS_BUBBLE_BUBBLE_MANAGER_H_