PlzNavigate: store POST data in the FrameNavigationEntry

content/common/page_state_serialization.cc

Index: content/common/page_state_serialization.cc
diff --git a/content/common/page_state_serialization.cc b/content/common/page_state_serialization.cc
index b9670c2f244bc8920ac245e90f2b26cc2924c70d..d9bfb9a2f5ce28c006d9e0fdabe425fa57d2cc73 100644
--- a/content/common/page_state_serialization.cc
+++ b/content/common/page_state_serialization.cc
@@ -683,6 +683,8 @@ ExplodedHttpBody::ExplodedHttpBody()
       is_null(true) {
 }
 
+ExplodedHttpBody::ExplodedHttpBody(const ExplodedHttpBody& other) = default;
+
 ExplodedHttpBody::~ExplodedHttpBody() {
 }
 
@@ -748,6 +750,18 @@ bool EncodePageState(const ExplodedPageState& exploded, std::string* encoded) {
   return true;
 }
 
+bool GeneratePostData(const ExplodedHttpBody exploded,

[Charlie Reis, 2016/04/27 23:00:55]

I'm not following why we need this.  Was this code somewhere else before?

[clamy, 2016/04/29 16:07:16]

We need it because we have code to generate a WebHTTPBody (which is a renderer
object), but we don't have code to generate the ResourceRequestBody (which is
the common object that can be used in the browser).

+                      ResourceRequestBody* http_body) {
+  if (exploded.is_null)
+    return false;
+
+  http_body->set_identifier(exploded.identifier);
+  for (auto element : exploded.elements)
+    http_body->AppendExplodedHTTPBodyElement(element);
+
+  return true;
+}
+
 #if defined(OS_ANDROID)
 bool DecodePageStateWithDeviceScaleFactorForTesting(
     const std::string& encoded,