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

content/common/navigation_params.h

Index: content/common/navigation_params.h
diff --git a/content/common/navigation_params.h b/content/common/navigation_params.h
index 86e5e06917fd0946b456f2b467a3790cfe58c223..260fdc73079297d30c2e617190c35c8e8a885aa8 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -42,6 +42,11 @@ struct CONTENT_EXPORT CommonNavigationParams {
                          const GURL& history_url_for_data_url);
   ~CommonNavigationParams();
 
+  // Helper function to determine if the navigation request to |url| should be
+  // sent to the network stack. It will not be sent for data URLs or JavaScript
+  // URLs, cases where no network request needs to be made.
+  static bool ShouldMakeNetworkRequest(const GURL& url);

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

A static method inside a struct strikes me as odd. I would rather have it
outside of any class/struct (it can remain in this file), with a name such as
ShouldMakeNetworkRequestForURL.

Also the comment should precise that this should ONLY be used with PlzNavigate.

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

Done.

+
   // The URL to navigate to.
   // PlzNavigate: May be modified when the navigation is ready to commit.
   GURL url;