PlzNavigate: transmit redirect info to the renderer side

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 c47da6b89c797ca8ceb40b93f578effdf10bbe92..105130740e309c150cf97dfef2ef918ad4a14e2d 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -22,6 +22,7 @@
 #include "content/public/common/referrer.h"
 #include "content/public/common/request_context_type.h"
 #include "content/public/common/resource_response.h"
+#include "net/url_request/redirect_info.h"
 #include "third_party/WebKit/public/platform/WebMixedContentContextType.h"
 #include "ui/base/page_transition_types.h"
 #include "url/gurl.h"
@@ -228,6 +229,8 @@ struct CONTENT_EXPORT RequestNavigationParams {
   RequestNavigationParams();
   RequestNavigationParams(bool is_overriding_user_agent,
                           const std::vector<GURL>& redirects,
+                          const GURL& original_url,
+                          const std::string& original_method,
                           bool can_load_local_resources,
                           const PageState& page_state,
                           int nav_entry_id,
@@ -254,6 +257,15 @@ struct CONTENT_EXPORT RequestNavigationParams {
   // The ResourceResponseInfos received during redirects.
   std::vector<ResourceResponseInfo> redirect_response;
 
+  // PlzNavigate
+  // The RedirectInfos received during redirects.
+  std::vector<net::RedirectInfo> redirect_infos;
+
+  // PlzNavigate
+  // The original URL & method for this navigation.
+  GURL original_url;
+  std::string original_method;

[nasko, 2017/02/10 23:53:09]

Can the method change between redirects? If not, why not reuse the one from
CommonNavigationParams?

[clamy, 2017/02/13 14:29:37]

Yes. It's possible for a POST (like a form submission for example) to be
redirected as a GET. See
https://cs.chromium.org/chromium/src/content/browser/frame_host/navigation_re...
(NavigationRequest::OnRequestRedirected) for where we update the method in
CommonNavigationParams.

[nasko, 2017/02/14 17:25:02]

Acknowledged.

+
   // Whether or not this url should be allowed to access local file://
   // resources.
   bool can_load_local_resources;