Bring RFH/RVH unit tests closer to reality of how RF/RV are initialized

content/test/test_render_view_host.cc

Index: content/test/test_render_view_host.cc
diff --git a/content/test/test_render_view_host.cc b/content/test/test_render_view_host.cc
index 0b7b8c105ad3dce383aabe8cdce92949d0a00922..d729fb6ef01dc20118149e978fe2e42c5b4109ab 100644
--- a/content/test/test_render_view_host.cc
+++ b/content/test/test_render_view_host.cc
@@ -246,6 +246,10 @@ bool TestRenderViewHost::CreateRenderView(
   set_renderer_initialized(true);
   DCHECK(IsRenderViewLive());
   opener_route_id_ = opener_route_id;
+  RenderFrameHost* main_frame = GetMainFrame();
+  if (main_frame)

[ncarter (slow), 2015/05/26 21:44:14]

When is this null? In RenderViewHostImpl's CreateRenderView it seems like this
call is done without checking for null.

[nasko, 2015/05/26 22:25:49]

This is in preparation for removing swapped out. When we want to create "swapped
out" object, in the next iteration we will be creating just a
RenderFrameProxyHost and the RenderViewHost will not have an associated main
frame.

+    static_cast<RenderFrameHostImpl*>(main_frame)->SetRenderFrameCreated(true);
+
   return true;
 }