Avert crash in ExtensionInstalledBubbleView::GetWindowTitle triggered

chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc

Index: chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc
diff --git a/chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc b/chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc
index 217db73ae8125f2eaad8b696d63117c4ccd5412f..1c85308ee2394f728e0cd74bfc864890f50e51d7 100644
--- a/chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc
+++ b/chrome/browser/ui/views/extensions/extension_installed_bubble_view.cc
@@ -171,6 +171,11 @@ void ExtensionInstalledBubbleView::CloseBubble() {
 }
 
 base::string16 ExtensionInstalledBubbleView::GetWindowTitle() const {

[Devlin, 2016/09/20 20:15:43]

Is there any way we could detect the view is closed, and if so, would this be
better handled higher up the chain (e.g. DialogDelegateView, View, etc)?  This
seems like something that could happen to any view rather than just this one.

+  // This may be called by screen readers after the window is closed.
+  // See crbug.com/648280
+  if (!controller_)
+    return base::string16();
+
   // Add the heading (for all options).
   base::string16 extension_name =
       base::UTF8ToUTF16(controller_->extension()->name());