Always create FrameTreeNodes and RenderFrameHosts for every frame.

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index bb6635ff536889f2b9591a30637e8079f5724f70..1d70fff727b5931fe3d61ca110e5355762a7e067 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -1382,8 +1382,9 @@ void RenderViewHostImpl::OnRenderProcessGone(int status, int exit_code) {
   render_view_termination_status_ =
       static_cast<base::TerminationStatus>(status);
 
-  // Reset state.
+  // Reset frame tree state.
   main_frame_id_ = -1;
+  delegate_->GetFrameTree()->SwapMainFrame(main_render_frame_host_.get());

[nasko, 2013/11/23 01:05:21]

This seems to do the right thing now, but I wonder if this is what we want
things to look long term. Maybe a TODO to revisit this once we have RFHM and
have a cleaner FrameTree Reset().

[Charlie Reis, 2013/11/23 01:20:13]

Good point.  SwapMainFrame is definitely going to be changing when
main_render_frame_host_ moves to RFHM.  I'll add a TODO.

 
   // Our base class RenderWidgetHost needs to reset some stuff.
   RendererExited(render_view_termination_status_, exit_code);