PlzNavigate: properly set the initiator of the navigation

content/common/navigation_params.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_COMMON_NAVIGATION_PARAMS_H_
 #define CONTENT_COMMON_NAVIGATION_PARAMS_H_
 
 #include <stdint.h>
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "content/common/content_export.h"
 #include "content/common/frame_message_enums.h"
 #include "content/common/resource_request_body_impl.h"
 #include "content/public/common/page_state.h"
 #include "content/public/common/referrer.h"
 #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 {
 
 // The LoFi state which determines whether to add the Lo-Fi header.
 enum LoFiState {
   // Let the browser process decide whether or not to request the Lo-Fi version.
   LOFI_UNSPECIFIED = 0,
 
   // Request a normal (non-Lo-Fi) version of the resource.
   LOFI_OFF,
@@ skipping 104 matching lines @@
 // PlzNavigate: parameters needed to start a navigation on the IO thread,
 // following a renderer-initiated navigation request.
 struct CONTENT_EXPORT BeginNavigationParams {
   // TODO(clamy): See if it is possible to reuse this in
   // ResourceMsg_Request_Params.
   BeginNavigationParams();
   BeginNavigationParams(std::string headers,
                         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.
   std::string headers;
 
   // net::URLRequest load flags (net::LOAD_NORMAL) by default).
   int load_flags;
 
   // True if the request was user initiated.
   bool has_user_gesture;
 
   // True if the ServiceWorker should be skipped.
   bool skip_service_worker;
 
   // 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 -----------------------------------------------------
 //
 // These structs are sent by the browser to the renderer to start/commit a
 // navigation depending on whether browser-side navigation is enabled.
 // Parameters used both in the current architecture and PlzNavigate should be
 // put in RequestNavigationParams.  Parameters only used by the current
 // architecture should go in StartNavigationParams.
 
@@ skipping 149 matching lines @@
   ~NavigationParams();
 
   CommonNavigationParams common_params;
   StartNavigationParams start_params;
   RequestNavigationParams request_params;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_NAVIGATION_PARAMS_H_