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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 1678303004: PlzNavigate: inform the renderer that a navigation is a POST (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 10 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
« no previous file with comments | « content/public/test/render_view_test.cc ('k') | content/renderer/render_view_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 482aa4929ae42835b2da7777c36de1672b72ba99..ec975b5e8bd90cff456118229e8b216f512d2e6c 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -526,6 +526,8 @@ WebURLRequest CreateURLRequestForNavigation(
request.setHTTPReferrer(web_referrer, common_params.referrer.policy);
}
+ request.setHTTPMethod(WebString::fromUTF8(common_params.method));
+
RequestExtraData* extra_data = new RequestExtraData();
extra_data->set_stream_override(std::move(stream_override));
extra_data->set_lofi_state(common_params.lofi_state);
@@ -601,7 +603,7 @@ CommonNavigationParams MakeCommonNavigationParams(
request->url(), referrer, extra_data->transition_type(),
FrameMsg_Navigate_Type::NORMAL, true, should_replace_current_entry,
ui_timestamp, report_type, GURL(), GURL(), extra_data->lofi_state(),
- base::TimeTicks::Now());
+ base::TimeTicks::Now(), request->httpMethod().latin1());
}
media::Context3D GetSharedMainThreadContext3D() {
@@ -5445,9 +5447,7 @@ void RenderFrameImpl::NavigateInternal(
}
}
- if (start_params.is_post && !browser_side_navigation) {
- request.setHTTPMethod(WebString::fromUTF8("POST"));
-
+ if (common_params.method == "POST" && !browser_side_navigation) {
// Set post data.
WebHTTPBody http_body;
http_body.initialize();
@@ -5738,11 +5738,11 @@ void RenderFrameImpl::BeginNavigation(blink::WebURLRequest* request,
Send(new FrameHostMsg_BeginNavigation(
routing_id_,
MakeCommonNavigationParams(request, should_replace_current_entry),
- BeginNavigationParams(
- request->httpMethod().latin1(), GetWebURLRequestHeaders(*request),
- GetLoadFlagsForWebURLRequest(*request), request->hasUserGesture(),
- request->skipServiceWorker(),
- GetRequestContextTypeForWebURLRequest(*request)),
+ BeginNavigationParams(GetWebURLRequestHeaders(*request),
+ GetLoadFlagsForWebURLRequest(*request),
+ request->hasUserGesture(),
+ request->skipServiceWorker(),
+ GetRequestContextTypeForWebURLRequest(*request)),
GetRequestBodyForWebURLRequest(*request)));
}
« no previous file with comments | « content/public/test/render_view_test.cc ('k') | content/renderer/render_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698