Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(172)

Unified Diff: content/browser/frame_host/navigation_entry_impl.cc

Issue 1956383003: Forwarding POST body into renderer after a cross-site transfer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Extracted a shared NavigationEntryImpl::ConstructResourceRequestBody. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/navigation_entry_impl.cc
diff --git a/content/browser/frame_host/navigation_entry_impl.cc b/content/browser/frame_host/navigation_entry_impl.cc
index 092281a6e51573406cd3598cad582d68b78097c3..f2d049da012749c8b0f11381c0f3ed97c3b8d7d5 100644
--- a/content/browser/frame_host/navigation_entry_impl.cc
+++ b/content/browser/frame_host/navigation_entry_impl.cc
@@ -18,6 +18,7 @@
#include "content/common/content_constants_internal.h"
#include "content/common/navigation_params.h"
#include "content/common/page_state_serialization.h"
+#include "content/common/resource_request_body.h"
#include "content/common/site_isolation_policy.h"
#include "content/public/common/content_constants.h"
#include "content/public/common/url_constants.h"
@@ -568,7 +569,21 @@ std::unique_ptr<NavigationEntryImpl> NavigationEntryImpl::CloneAndReplace(
return copy;
}
+scoped_refptr<ResourceRequestBody>
+NavigationEntryImpl::ConstructResourceRequestBody() const {
+ scoped_refptr<ResourceRequestBody> browser_initiated_post_body;
+ if (GetHasPostData()) {
+ if (const base::RefCountedMemory* memory = GetBrowserInitiatedPostData()) {
+ browser_initiated_post_body = new ResourceRequestBody();
+ browser_initiated_post_body->AppendBytes(memory->front_as<char>(),
+ memory->size());
+ }
+ }
+ return browser_initiated_post_body;
+}
+
CommonNavigationParams NavigationEntryImpl::ConstructCommonNavigationParams(
+ const std::string& method,
const GURL& dest_url,
const Referrer& dest_referrer,
FrameMsg_Navigate_Type::Value navigation_type,
@@ -583,29 +598,25 @@ CommonNavigationParams NavigationEntryImpl::ConstructCommonNavigationParams(
ui_timestamp = intent_received_timestamp();
#endif
+ if (GetHasPostData())
+ DCHECK_EQ("POST", method);
+
return CommonNavigationParams(
dest_url, dest_referrer, GetTransitionType(), navigation_type,
!IsViewSourceMode(), should_replace_entry(), ui_timestamp, report_type,
GetBaseURLForDataURL(), GetHistoryURLForDataURL(), lofi_state,
- navigation_start, GetHasPostData() ? "POST" : "GET");
+ navigation_start, method);
}
-StartNavigationParams NavigationEntryImpl::ConstructStartNavigationParams()
- const {
- std::vector<unsigned char> browser_initiated_post_data;
- if (GetBrowserInitiatedPostData()) {
- browser_initiated_post_data.assign(
- GetBrowserInitiatedPostData()->front(),
- GetBrowserInitiatedPostData()->front() +
- GetBrowserInitiatedPostData()->size());
- }
-
- return StartNavigationParams(extra_headers(), browser_initiated_post_data,
+StartNavigationParams NavigationEntryImpl::ConstructStartNavigationParams(
+ const scoped_refptr<ResourceRequestBody>& post_body) const {
+ return StartNavigationParams(
+ extra_headers(), post_body ? post_body : ConstructResourceRequestBody(),
#if defined(OS_ANDROID)
- has_user_gesture(),
+ has_user_gesture(),
#endif
- transferred_global_request_id().child_id,
- transferred_global_request_id().request_id);
+ transferred_global_request_id().child_id,
+ transferred_global_request_id().request_id);
}
RequestNavigationParams NavigationEntryImpl::ConstructRequestNavigationParams(

Powered by Google App Engine
This is Rietveld 408576698