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

content/common/navigation_params.cc

Index: content/common/navigation_params.cc
diff --git a/content/common/navigation_params.cc b/content/common/navigation_params.cc
index 8b70c8ffb9718acfc22352f71d3643591cad57c7..d57b036fb29325ba6e9c9e76897d4040ad67aac6 100644
--- a/content/common/navigation_params.cc
+++ b/content/common/navigation_params.cc
@@ -8,6 +8,14 @@
 
 namespace content {
 
+// static
+bool CommonNavigationParams::ShouldMakeNetworkRequest(const GURL& url) {
+  // Data and Javascript urls should not make network requests.
+  // TODO(clamy): same document navigations should not make network requests.

[clamy, 2015/07/03 11:41:44]

Please add a DCHECK that browser-side navigation is enabled. We don't want other
people to use it right now.

[Fabrice (no longer in Chrome), 2015/07/03 14:55:35]

Similar checks in the code are CHECKS and not DCHECKS so I used that instead.
Done.

+  return !url.SchemeIs(url::kDataScheme) && url != GURL(url::kAboutBlankURL) &&
+         !url.SchemeIs(url::kJavaScriptScheme);
+}
+
 CommonNavigationParams::CommonNavigationParams()
     : transition(ui::PAGE_TRANSITION_LINK),
       navigation_type(FrameMsg_Navigate_Type::NORMAL),