Fix wrong initialization of renderer_initiated_creation parameter.

chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc

Index: chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc
diff --git a/chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc b/chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc
index ce0c9c755cf48f4c2b7b64af1cfaca3f63a23740..f71ccff602d456ceec8eedd2470a1cf37f49f16a 100644
--- a/chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc
+++ b/chrome/browser/captive_portal/captive_portal_tab_helper_unittest.cc
@@ -76,6 +76,10 @@ class CaptivePortalTabHelperTest : public ChromeRenderViewHostTestHarness {
     ChromeRenderViewHostTestHarness::SetUp();
     web_contents1_.reset(CreateTestWebContents());
     web_contents2_.reset(CreateTestWebContents());
+    content::RenderFrameHostTester::For(main_render_frame1())
+        ->InitializeRenderFrameIfNeeded();
+    content::RenderFrameHostTester::For(main_render_frame2())
+        ->InitializeRenderFrameIfNeeded();

[mmenke, 2015/06/08 16:09:49]

Need to include the header for RenderFrameHostTester.

[mmenke, 2015/06/08 16:09:50]

Why are these needed?  What do they do?  I assume they aren't needed for all
TestWebContents, or this could just be part of CreateTestWebContents().

[lfg, 2015/06/08 17:04:01]

The header is already included because of the inheritance (it is defined in
test_renderer_host.h, which defines RenderViewHostTestHarness, which is the
superclass of ChromeRenderViewHostTestHarness).

I can include it explicitly if you prefer.

[lfg, 2015/06/08 17:04:01]

There are a few reasons for this, the main one is that we're adding extra
asserts to make the code more robust and initializing the RenderFrame is
something that needs to be done before we can use it. In a real-world scenario,
creating a WebContents doesn't initialize the RenderFrame. We can't expect to
receive IPCs without initializing it first (i.e. DidStartProvisionalLoadForFrame
comes from an IPC from a renderer).

[mmenke, 2015/06/08 17:15:06]

Could you add some sort of comment here?  It's neither obvious from this method
or the RenderFrameHostTester header what's going on.  Just something like
"WebContents create no main frame on cosntruction, and these tests expect one to
exist."

Same goes for the browser/net file.  Alternatively, could mention something
about it in web_contents_tester.h

[mmenke, 2015/06/08 17:15:06]

I'd prefer it be included explicitly - same goes for the browser/net file.

[lfg, 2015/06/08 18:25:51]

Done.

[lfg, 2015/06/08 18:25:51]

Done.

   }
 
   void TearDown() override {