PlzNavigate: stop navigations when opening a document for write

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 105130740e309c150cf97dfef2ef918ad4a14e2d..c635a2c1e1d4e8a6da4639aaf3ee59c2596cf850 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -142,7 +142,8 @@ struct CONTENT_EXPORT BeginNavigationParams {
       bool skip_service_worker,
       RequestContextType request_context_type,
       blink::WebMixedContentContextType mixed_content_context_type,
-      const base::Optional<url::Origin>& initiator_origin);
+      const base::Optional<url::Origin>& initiator_origin,
+      int renderer_navigation_id);
   BeginNavigationParams(const BeginNavigationParams& other);
   ~BeginNavigationParams();
 
@@ -176,6 +177,11 @@ struct CONTENT_EXPORT BeginNavigationParams {
   // If the transition type is a client side redirect, then this holds the URL
   // of the page that had the client side redirect.
   GURL client_side_redirect_url;
+
+  // ID provided by the renderer to identify this navigation. It will be sent
+  // back to the renderer if the browser decides to drop the navigation.
+  // -1 is this navigation wasn't renderer-initiated.

[nasko, 2017/03/16 21:23:14]

nit: "Value of -1 means this ..."

Why not define a constant in this header, something like kInvalidNavigationId,
so it can be used in the rest of the codebase? It is more meaningful to read the
constant name than -1 and less error prone.

[clamy, 2017/03/17 15:19:11]

Done.

+  int renderer_navigation_id;
 };
 
 // Provided by the browser -----------------------------------------------------