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

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 d2f83e756c13622554047902bcdc63d4c24c164b..325785717ca1260d18bc7b7fbf586c027750c099 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -9,10 +9,12 @@
 
 #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.h"
 #include "content/public/common/page_state.h"
 #include "content/public/common/referrer.h"
 #include "content/public/common/request_context_type.h"
@@ -172,23 +174,21 @@ struct CONTENT_EXPORT BeginNavigationParams {
 // PlzNavigate: These are not used.
 struct CONTENT_EXPORT StartNavigationParams {
   StartNavigationParams();
-  StartNavigationParams(
-      const std::string& extra_headers,
-      const std::vector<unsigned char>& browser_initiated_post_data,
+  StartNavigationParams(const std::string& extra_headers,
+                        const scoped_refptr<ResourceRequestBody>& post_data,
 #if defined(OS_ANDROID)
-      bool has_user_gesture,
+                        bool has_user_gesture,
 #endif
-      int transferred_request_child_id,
-      int transferred_request_request_id);
+                        int transferred_request_child_id,
+                        int transferred_request_request_id);
   StartNavigationParams(const StartNavigationParams& other);
   ~StartNavigationParams();
 
   // Extra headers (separated by \n) to send during the request.
   std::string extra_headers;
 
-  // If is_post is true, holds the post_data information from browser. Empty
-  // otherwise.
-  std::vector<unsigned char> browser_initiated_post_data;
+  // If is_post is true, holds the post data.
+  scoped_refptr<ResourceRequestBody> post_data;

[clamy, 2016/05/12 05:32:48]

Note: following my CL, this may end up in CommonNavigationParams. In any case,
since it will be used both by vanilla navigations and PlzNavigate, it shouldn't
be in the vanilla specific params.

Also, I'm a bit worried about replacing browser_initiated_post_data. If you do
that, you should properly set it up when creating the StartNavigationParams.

[Łukasz Anforowicz, 2016/05/17 23:52:56]

In your CL [1] this ends up in PlzNavigate-specific IPC message -
FrameMsg_CommitNavigation.  Please let me know if you'd like me to tweak this CL
and move |post_data| to another location (shared by PlzNavigate?).

[1] https://codereview.chromium.org/1907443006/patch/200001/210010
It seemed that ResourceRequestBody is a generalization of
browser_initiated_post_data.  In most recent patchsets it should be properly
initialized.

 
 #if defined(OS_ANDROID)
   bool has_user_gesture;