Forwarding POST body into renderer after a cross-site transfer.

content/browser/frame_host/navigation_handle_impl.h

 // Copyright 2015 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_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
 
 #include "content/public/browser/navigation_handle.h"
 
 #include <stddef.h>
 
 #include "base/callback.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"
 #include "base/memory/scoped_vector.h"
 #include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/navigation_data.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "url/gurl.h"
 
 struct FrameHostMsg_DidCommitProvisionalLoad_Params;
 
 namespace content {
 
 class NavigatorDelegate;
+class ResourceRequestBody;
 class ServiceWorkerContextWrapper;
 class ServiceWorkerNavigationHandle;
 struct NavigationRequestInfo;
 
 // This class keeps track of a single navigation. It is created upon receipt of
 // a DidStartProvisionalLoad IPC in a RenderFrameHost. The RenderFrameHost owns
 // the newly created NavigationHandleImpl as long as the navigation is ongoing.
 // The NavigationHandleImpl in the RenderFrameHost will be reset when the
 // navigation stops, that is if one of the following events happen:
 //   - The RenderFrameHost receives a DidStartProvisionalLoad IPC for a new
@@ skipping 112 matching lines @@
   void set_is_transferring(bool is_transferring) {
     is_transferring_ = is_transferring;
   }
 
   // Updates the RenderFrameHost that is about to commit the navigation. This
   // is used during transfer navigations.
   void set_render_frame_host(RenderFrameHostImpl* render_frame_host) {
     render_frame_host_ = render_frame_host;
   }
 
+  // Gets post body associated with this navigation.
+  const scoped_refptr<ResourceRequestBody>& resource_request_body() const {
+    return resource_request_body_;
+  }
+
+  // Gets HTTP method associated with this navigation.
+  const std::string& method() const { return method_; }

[clamy, 2016/05/23 15:32:28]

Suggestion: should we change the public interface method IsPost into GetMethod?

[Łukasz Anforowicz, 2016/05/23 16:25:10]

Yes - this seems like the right thing to do - only considering GET and POST (and
ignoring PUT and other HTTP methods) seems fishy and therefore GetMethod seems
better than IsPost.

BTW: After your suggestion I've also realized that like other similar accessors,
GetMethod should also verify CHECK_NE(INITIAL, state_).

Done.

+
   // PlzNavigate
   void InitServiceWorkerHandle(
       ServiceWorkerContextWrapper* service_worker_context);
   ServiceWorkerNavigationHandle* service_worker_handle() const {
     return service_worker_handle_.get();
   }
 
   typedef base::Callback<void(NavigationThrottle::ThrottleCheckResult)>
       ThrottleChecksFinishedCallback;
 
   // Called when the URLRequest will start in the network stack.  |callback|
   // will be called when all throttle checks have completed. This will allow
   // the caller to cancel the navigation or let it proceed.
-  void WillStartRequest(const std::string& method,
-                        const Referrer& sanitized_referrer,
-                        bool has_user_gesture,
-                        ui::PageTransition transition,
-                        bool is_external_protocol,
-                        const ThrottleChecksFinishedCallback& callback);
+  void WillStartRequest(
+      const std::string& method,
+      const scoped_refptr<content::ResourceRequestBody>& resource_request_body,
+      const Referrer& sanitized_referrer,
+      bool has_user_gesture,
+      ui::PageTransition transition,
+      bool is_external_protocol,
+      const ThrottleChecksFinishedCallback& callback);
 
   // Called when the URLRequest will be redirected in the network stack.
   // |callback| will be called when all throttles check have completed. This
   // will allow the caller to cancel the navigation or let it proceed.
   // This will also inform the delegate that the request was redirected.
   void WillRedirectRequest(
       const GURL& new_url,
       const std::string& new_method,
       const GURL& new_referrer_url,
       bool new_is_external_protocol,
@@ skipping 82 matching lines @@
   const bool is_renderer_initiated_;
   bool is_same_page_;
   const bool is_synchronous_;
   const bool is_srcdoc_;
   bool was_redirected_;
   scoped_refptr<net::HttpResponseHeaders> response_headers_;
 
   // The HTTP method used for the navigation.
   std::string method_;
 
+  // The body of the request (i.e. body of a post request).
+  scoped_refptr<ResourceRequestBody> resource_request_body_;
+
   // The state the navigation is in.
   State state_;
 
   // Whether the navigation is in the middle of a transfer. Set to false when
   // the DidStartProvisionalLoad is received from the new renderer.
   bool is_transferring_;
 
   // The FrameTreeNode this navigation is happening in.
   FrameTreeNode* frame_tree_node_;
 
@@ skipping 19 matching lines @@
 
   // Embedder data tied to this navigation.
   std::unique_ptr<NavigationData> navigation_data_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_