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

chrome/browser/ui/extensions/extension_installed_bubble.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/ui/extensions/extension_installed_bubble.h"
 
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 194 matching lines @@
 
 bool ExtensionInstalledBubble::ShouldClose(BubbleCloseReason reason) const {
   // Installing an extension triggers a navigation event that should be ignored.
   return reason != BUBBLE_CLOSE_NAVIGATED;
 }
 
 std::string ExtensionInstalledBubble::GetName() const {
   return "ExtensionInstalled";
 }
 
+const content::RenderFrameHost* ExtensionInstalledBubble::OwningFrame() const {
+  return nullptr;

[Peter Kasting, 2016/02/06 04:13:08]

I don't know anything about this bubble, so I just want to make sure that
returning null here is safe.

[hcarmona, 2016/02/09 00:31:05]

I worked on this bubble. Returning null is fine here because this bubble
isn't related to anything in the render view.

+}
+
 base::string16 ExtensionInstalledBubble::GetHowToUseDescription() const {
   int message_id = 0;
   base::string16 extra;
   if (action_command_)
     extra = action_command_->accelerator().GetShortcutText();
 
   switch (type_) {
     case BROWSER_ACTION:
       message_id = extra.empty() ? IDS_EXTENSION_INSTALLED_BROWSER_ACTION_INFO :
           IDS_EXTENSION_INSTALLED_BROWSER_ACTION_INFO_WITH_SHORTCUT;
@@ skipping 63 matching lines @@
       break;
     case OMNIBOX_KEYWORD:
       options_ |= HOW_TO_USE | HOW_TO_MANAGE;
       anchor_position_ = ANCHOR_OMNIBOX;
       break;
     case GENERIC:
       anchor_position_ = ANCHOR_APP_MENU;
       break;
   }
 }