PlzNavigate: Do not send synchronous navigations from the renderer to the browser.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 2e7d77a987884907027c9592b092779f2be946ff..13f299fd243d25586d91c071678e940faa1dab63 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4329,10 +4329,11 @@ WebNavigationPolicy RenderFrameImpl::DecidePolicyForNavigation(
     return blink::WebNavigationPolicyIgnore;
   }
 
-  // PlzNavigate: send the request to the browser if needed.
+  // PlzNavigate: if the navigation is not synchronous, send it to the browser.

[nasko, 2015/07/06 11:06:16]

The comment talks about synchronous navigations, yet the name is about network
requests. I think we should try to come up with a bit better naming for this.

[Fabrice (no longer in Chrome), 2015/07/06 12:45:41]

Updated this comment too, WDYT?

   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableBrowserSideNavigation) &&
-      info.urlRequest.checkForBrowserSideNavigation()) {
+      info.urlRequest.checkForBrowserSideNavigation() &&
+      ShouldMakeNetworkRequestForURL(url)) {
     BeginNavigation(&info.urlRequest);
     return blink::WebNavigationPolicyIgnore;
   }