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..a38e5c89b974c7fb9bbfcaefc1a4a956d54db8e9 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;
+  // Body of HTTP POST request.
+  scoped_refptr<ResourceRequestBody> post_data;

[clamy, 2016/05/20 15:49:14]

Actually, I think this should be part of CommonNavigationParams. Right now, we
send it in the BeginNavigation IPC, the CommitNavigation IPC, and your CL adds
it to be used through the StartNavigationParams in the Navigate IPC. Let's have
it be a member of CommonNavigationParams (which is sent in all those IPCs), and
remove the extra member of FramehostMsg_BeginNavigation and
FrameMsg_CommitNavigation.

[Łukasz Anforowicz, 2016/05/20 22:18:47]

Done.  I've done that initially here, but then decided to move this refactoring
to a separate CL: https://codereview.chromium.org/1999943002/

 
 #if defined(OS_ANDROID)
   bool has_user_gesture;