PlzNavigate: store POST data in the FrameNavigationEntry

content/browser/frame_host/frame_navigation_entry.cc

 // 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.
 
 #include "content/browser/frame_host/frame_navigation_entry.h"
 
 #include <utility>
 
+#include "content/common/page_state_serialization.h"
+#include "content/common/site_isolation_policy.h"
+
 namespace content {
 
 FrameNavigationEntry::FrameNavigationEntry()
     : item_sequence_number_(-1), document_sequence_number_(-1), post_id_(-1) {}
 
 FrameNavigationEntry::FrameNavigationEntry(
     const std::string& frame_unique_name,
     int64_t item_sequence_number,
     int64_t document_sequence_number,
     scoped_refptr<SiteInstanceImpl> site_instance,
@@ skipping 54 matching lines @@
   // location.replace is classified as NEW_PAGE rather than EXISTING_PAGE.
   // Same for document sequence number.  See https://crbug.com/596707.
   item_sequence_number_ = item_sequence_number;
 }
 
 void FrameNavigationEntry::set_document_sequence_number(
     int64_t document_sequence_number) {
   document_sequence_number_ = document_sequence_number;
 }
 
+scoped_refptr<ResourceRequestBody> FrameNavigationEntry::GetPostData() const {
+  DCHECK(SiteIsolationPolicy::UseSubframeNavigationEntries());
+  scoped_refptr<ResourceRequestBody> body;
+  if (method_ != "POST")
+    return body;

[carlosk, 2016/05/20 08:59:47]

nit: it seems clearer to return nullptr here and below and only create the
scoped_refptr in line 97.

[clamy, 2016/05/20 11:00:12]

Done.

+
+  // Generate the body from the PageState.
+  ExplodedPageState exploded_state;
+  if (!DecodePageState(page_state_.ToEncodedData(), &exploded_state))
+    return body;
+
+  body = new ResourceRequestBody();
+  if (!GeneratePostData(exploded_state.top.http_body, body.get()))
+    return nullptr;
+
+  return body;
+}
+
 }  // namespace content