Index: content/browser/frame_host/navigator_impl.cc |
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc |
index 6fcdb074b4a0f6badd1ea04256214a7e2223480d..f969753ba73ba6b82e98e3d35ff6a5ae5620f48c 100644 |
--- a/content/browser/frame_host/navigator_impl.cc |
+++ b/content/browser/frame_host/navigator_impl.cc |
@@ -257,7 +257,8 @@ bool NavigatorImpl::NavigateToEntry( |
const NavigationEntryImpl& entry, |
NavigationController::ReloadType reload_type, |
bool is_same_document_history_load, |
- bool is_pending_entry) { |
+ bool is_pending_entry, |
+ const scoped_refptr<ResourceRequestBody>& post_body) { |
TRACE_EVENT0("browser,navigation", "NavigatorImpl::NavigateToEntry"); |
GURL dest_url = frame_entry.url(); |
@@ -375,10 +376,10 @@ bool NavigatorImpl::NavigateToEntry( |
FrameMsg_Navigate_Type::Value navigation_type = GetNavigationType( |
controller_->GetBrowserContext(), entry, reload_type); |
dest_render_frame_host->Navigate( |
- entry.ConstructCommonNavigationParams(dest_url, dest_referrer, |
- navigation_type, lofi_state, |
- navigation_start), |
- entry.ConstructStartNavigationParams(), |
+ entry.ConstructCommonNavigationParams(frame_entry.method(), dest_url, |
+ dest_referrer, navigation_type, |
+ lofi_state, navigation_start), |
+ entry.ConstructStartNavigationParams(post_body), |
entry.ConstructRequestNavigationParams( |
frame_entry, is_same_document_history_load, |
frame_tree_node->has_committed_real_load(), |
@@ -424,7 +425,7 @@ bool NavigatorImpl::NavigateToPendingEntry( |
bool is_same_document_history_load) { |
return NavigateToEntry(frame_tree_node, frame_entry, |
*controller_->GetPendingEntry(), reload_type, |
- is_same_document_history_load, true); |
+ is_same_document_history_load, true, nullptr); |
} |
bool NavigatorImpl::NavigateNewChildFrame( |
@@ -445,7 +446,7 @@ bool NavigatorImpl::NavigateNewChildFrame( |
return NavigateToEntry(render_frame_host->frame_tree_node(), *frame_entry, |
*entry, NavigationControllerImpl::NO_RELOAD, false, |
- false); |
+ false, nullptr); |
} |
void NavigatorImpl::DidNavigate( |
@@ -707,7 +708,8 @@ void NavigatorImpl::RequestTransferURL( |
const Referrer& referrer, |
ui::PageTransition page_transition, |
const GlobalRequestID& transferred_global_request_id, |
- bool should_replace_current_entry) { |
+ bool should_replace_current_entry, |
+ const scoped_refptr<ResourceRequestBody>& post_body) { |
// This call only makes sense for subframes if OOPIFs are possible. |
DCHECK(!render_frame_host->GetParent() || |
SiteIsolationPolicy::AreCrossProcessFramesPossible()); |
@@ -771,12 +773,10 @@ void NavigatorImpl::RequestTransferURL( |
is_renderer_initiated, std::string(), |
controller_->GetBrowserContext())); |
} |
- // TODO(creis): Handle POST submissions. See https://crbug.com/582211 and |
- // https://crbug.com/101395. |
entry->AddOrUpdateFrameEntry( |
node, -1, -1, nullptr, |
static_cast<SiteInstanceImpl*>(source_site_instance), dest_url, |
- referrer_to_use, PageState(), "GET", -1); |
+ referrer_to_use, PageState(), post_body ? "POST" : "GET", -1); |
} else { |
// Main frame case. |
entry = NavigationEntryImpl::FromNavigationEntry( |
@@ -806,14 +806,13 @@ void NavigatorImpl::RequestTransferURL( |
// further in https://crbug.com/536906. |
scoped_refptr<FrameNavigationEntry> frame_entry(entry->GetFrameEntry(node)); |
if (!frame_entry) { |
- // TODO(creis): Handle POST submissions here, as above. |
frame_entry = new FrameNavigationEntry( |
node->unique_name(), -1, -1, nullptr, |
static_cast<SiteInstanceImpl*>(source_site_instance), dest_url, |
- referrer_to_use, "GET", -1); |
+ referrer_to_use, post_body ? "POST" : "GET", -1); |
} |
NavigateToEntry(node, *frame_entry, *entry.get(), |
- NavigationController::NO_RELOAD, false, false); |
+ NavigationController::NO_RELOAD, false, false, post_body); |
} |
// PlzNavigate |