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 RenderFrameHostTest
mmenke
2015/06/08 16:09:50
Why are these needed? What do they do? I assume
lfg
2015/06/08 17:04:01
The header is already included because of the inhe
lfg
2015/06/08 17:04:01
There are a few reasons for this, the main one is
mmenke
2015/06/08 17:15:06
Could you add some sort of comment here? It's nei
mmenke
2015/06/08 17:15:06
I'd prefer it be included explicitly - same goes f
lfg
2015/06/08 18:25:51
Done.
lfg
2015/06/08 18:25:51
Done.
|
} |
void TearDown() override { |