Null-check GetView before using it in GetFocusedRenderWidgetHost.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 18589f15e14ca89b2a2c84366110b23d7f66114f..77bb05a7d55c8633b863ed15219a714c1677de49 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1647,8 +1647,13 @@ RenderWidgetHostImpl* WebContentsImpl::GetFocusedRenderWidgetHost(
   if (!focused_frame)
     return receiving_widget;
 
-  return RenderWidgetHostImpl::From(
-      focused_frame->current_frame_host()->GetView()->GetRenderWidgetHost());
+  // The view may be null if a subframe's renderer process has crashed while
+  // the subframe has focus.  Give the event to the main frame in this case.

[Charlie Reis, 2015/12/07 22:47:24]

Sanity check: What will the main frame do with it?  Will it recognize that none
of its frames had focus and thus do nothing with it?  Or will the event go to
something in the main frame?

I worry about a user typing something long into a subframe, and half of it
unexpectedly going into a different form in the main frame if the subframe
crashes while they're typing.

[alexmos, 2015/12/07 23:12:59]

That's a good point.  It's going to go to the main frame.  I've tried this out,
and if you start typing into field A on the main frame, then focus field B in a
subframe and start typing, and then crash the subframe renderer, keyboard input
from that point on will go into field A on the main frame.  I agree this
behavior would be unexpected/dangerous.  

I could return nullptr instead of the main frame here, though I'd need to also
update the three call sites for GetFocusedRWH to also do a null-check and drop
the event in this case.  Does this plan sound good?

[Charlie Reis, 2015/12/07 23:16:52]

Yes, let's do that.  Thanks!

[alexmos, 2015/12/07 23:21:50]

Done.  Please take a look.

+  RenderWidgetHostView* view = focused_frame->current_frame_host()->GetView();
+  if (!view)
+    return receiving_widget;
+
+  return RenderWidgetHostImpl::From(view->GetRenderWidgetHost());
 }
 
 void WebContentsImpl::EnterFullscreenMode(const GURL& origin) {