| 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
|
|
|