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..ac38f1654276553556fb70fe5ee93a3f23ec6436 100644
--- a/content/test/test_web_contents.h
+++ b/content/test/test_web_contents.h
@@ -94,6 +94,12 @@ class TestWebContents : public WebContentsImpl, public WebContentsTester {
                                 int error_code,
                                 const base::string16& error_description);
 
+  // Returns the same result as a call to GetPendingMainFrame.

[nasko, 2015/01/06 00:03:18]

nit: If it returns the same result, then we don't need it, do we? Let's add
something like " when running in regular mode."

[carlosk, 2015/01/08 16:05:56]

Updated the comment in that sense.

+  // PlzNavigate: when browser side navigation is enabled it returns the
+  // speculative RenderFrameHost for the main frame if one exists. Otherwise
+  // returns nullptr.
+  TestRenderFrameHost* GetFutureMainFrame() const;

[nasko, 2015/01/06 00:03:18]

nit: I'm not a big fan of "Future" as it implies some sort of promise this will
be the RFH. Sadly I don't have a much better suggestion either :(. "Potential"
is arguably worse.

[carlosk, 2015/01/08 16:05:56]

Well then... :)

I changed it to GetNextMainFrameCandidate that I hope is better.

[nasko, 2015/01/08 23:36:37]

Yes, I like this better! Thanks!

+
  protected:
   // The deprecated WebContentsTester still needs to subclass this.
   explicit TestWebContents(BrowserContext* browser_context);