PlzNavigate: Speculatively spawns a renderer process for navigations.

content/test/test_web_contents.h

Index: content/test/test_web_contents.h
diff --git a/content/test/test_web_contents.h b/content/test/test_web_contents.h
index 8e3c2b55f382e97046ed300a1fbcd5cd4df6e19a..7c10013093fbf4f9bc7ceb30853a998e856b5bf1 100644
--- a/content/test/test_web_contents.h
+++ b/content/test/test_web_contents.h
@@ -117,6 +117,16 @@ class TestWebContents : public WebContentsImpl, public WebContentsTester {
   void ShowCreatedWidget(int route_id, const gfx::Rect& initial_pos) override;
   void ShowCreatedFullscreenWidget(int route_id) override;
 
+  // PlzNavigate
+  // Gets the speculative RenderFrameHost, if any, for the main frame.
+  TestRenderFrameHost* GetSpeculativeMainFrame() const;

[clamy, 2014/12/29 17:36:22]

This method is currently private and used exactly one time in
test_web_contents.cc and one line long. Please remove it if it is not going to
be used publicly or make it public if it will be used.

[carlosk, 2014/12/30 15:54:50]

Done.

+
+  // PlzNavigate
+  // Gets the whether the pending or the speculative RenderFrameHost, whichever
+  // makes sense for the current environment. Returns nullptr if the selected
+  // RenderFrameHost is not set.
+  TestRenderFrameHost* GetFutureMainFrame() const;

[clamy, 2014/12/29 17:36:22]

This method should not be private as we expect to use it in several unit tests.
Also please rephrase the comment with a description of what the method does in
the current code path, and then on a new line PlzNavigate: with a description of
what it does in PlzNavigate case.

[carlosk, 2014/12/30 15:54:50]

Done.

+
   RenderViewHostDelegateView* delegate_view_override_;
 
   // Expectations for arguments of |SetHistoryOffsetAndLength()|.