Make to call WebContentsImpl::RenderViewCreated() when we create child window.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
===================================================================
--- content/browser/web_contents/web_contents_impl.cc	(revision 256801)
+++ content/browser/web_contents/web_contents_impl.cc	(working copy)
@@ -1377,6 +1377,7 @@
   if (params.disposition == NEW_BACKGROUND_TAB)
     create_params.initially_hidden = true;
   new_contents->Init(create_params);
+  new_contents->RenderViewCreated(new_contents->GetRenderViewHost());

[sky, 2014/03/19 15:41:31]

This should be invoked from RenderViewHostImpl. Is there a reason it isn't
happening here?

[Charlie Reis, 2014/03/19 23:31:59]

Agreed, this is not the right place for this call.  Please do not land this.

If you're not seeing it called, it sounds like you may be missing the call to
RenderViewHostImpl::CreateRenderView.

 
   // Save the window for later if we're not suppressing the opener (since it
   // will be shown immediately).