PlzNavigate: properly set the initiator of the navigation

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 1a09b3332381818ea92a7535aaac82bd37d081eb..23d62a115e6ddb8cbf6b1f44ffc229976fca9b77 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -20,6 +20,7 @@
 #include "content/public/common/request_context_type.h"
 #include "ui/base/page_transition_types.h"
 #include "url/gurl.h"
+#include "url/origin.h"
 
 namespace content {
 
@@ -144,7 +145,8 @@ struct CONTENT_EXPORT BeginNavigationParams {
                         int load_flags,
                         bool has_user_gesture,
                         bool skip_service_worker,
-                        RequestContextType request_context_type);
+                        RequestContextType request_context_type,
+                        const url::Origin& initiator);
   BeginNavigationParams(const BeginNavigationParams& other);
 
   // Additional HTTP request headers.
@@ -161,6 +163,11 @@ struct CONTENT_EXPORT BeginNavigationParams {
 
   // Indicates the request context type.
   RequestContextType request_context_type;
+
+  // Indicates the initiator of the request. Normally this is the origin of the
+  // url of the navigation. In auxilliary navigations, this is the origin of

[nasko, 2016/06/28 18:32:23]

Really? I always thought of initiator as the current document that triggered the
navigation, not the URL we are navigating to.

[clamy, 2016/06/29 11:23:20]

I don't know. I followed the comments of the
RequestDataResourceDispatcherHostBrowserTests
(https://cs.chromium.org/chromium/src/content/browser/loader/resource_dispatch...).

+  // the document that triggered the navigation.
+  url::Origin initiator;
 };
 
 // Provided by the browser -----------------------------------------------------