PlzNavigate: track pending commits

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 4d9b3ab9967e7d186c5838002b50209fc48a165d..a03d112bbf8dbc10e96ff429e38bff1782760c68 100644
--- a/content/test/test_web_contents.h
+++ b/content/test/test_web_contents.h
@@ -48,10 +48,8 @@ class TestWebContents : public WebContentsImpl, public WebContentsTester {
                                    const Referrer& referrer,
                                    ui::PageTransition transition) override;
 
-  // State accessor.
-  bool cross_navigation_pending() {
-    return GetRenderManager()->cross_navigation_pending_;
-  }
+  // True if a cross-site navigation is pending.
+  bool CrossNavigationPending();

[nasko, 2015/04/15 21:12:46]

Shouldn't it be CrossSiteNavigationPending, since the navigation is cross-site,
not just "cross"?

[Charlie Reis, 2015/04/15 21:54:17]

Yes, as long as we're updating the name, please fix this.  (It's always bothered
me.)  I think CrossProcessNavigationPending is probably better, since not all
cross-site navigations are cross-process.  (Technically it could end up in a new
SiteInstance but the same RenderProcessHost, but that's splitting hairs.)

(As I mentioned separately, I'll remove that cross_navigation_pending_ member
once this CL lands.)

[clamy, 2015/04/16 13:18:36]

Done.

 
   // Prevent interaction with views.
   bool CreateRenderViewForRenderManager(RenderViewHost* render_view_host,