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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 2419093002: Revert of Preserving Content-Type header from http request in OpenURL path. (Closed)
Patch Set: Manually resolved conflicts with r425338 in navigator_impl.cc. Created 4 years, 2 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/renderer/render_frame_impl.h ('k') | content/renderer/render_frame_proxy.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 88a5cd0dd296d70986ff181ec20396e4fa53c9fb..462be0921a400456cebf0b598c7f44fe1a9b6ac2 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3163,8 +3163,7 @@ void RenderFrameImpl::loadURLExternally(const blink::WebURLRequest& request,
suggested_name));
} else {
OpenURL(request.url(), IsHttpPost(request),
- GetRequestBodyForWebURLRequest(request),
- GetWebURLRequestHeaders(request), referrer, policy,
+ GetRequestBodyForWebURLRequest(request), referrer, policy,
should_replace_current_entry, false);
}
}
@@ -5094,8 +5093,7 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
render_view_->renderer_preferences_
.browser_handles_all_top_level_requests) {
OpenURL(url, IsHttpPost(info.urlRequest),
- GetRequestBodyForWebURLRequest(info.urlRequest),
- GetWebURLRequestHeaders(info.urlRequest), referrer,
+ GetRequestBodyForWebURLRequest(info.urlRequest), referrer,
info.defaultPolicy, info.replacesCurrentHistoryItem, false);
return blink::WebNavigationPolicyIgnore; // Suppress the load here.
}
@@ -5115,8 +5113,7 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
// JavaScript on the page is trying to interrupt the history navigation.
if (!info.isClientRedirect) {
OpenURL(url, IsHttpPost(info.urlRequest),
- GetRequestBodyForWebURLRequest(info.urlRequest),
- GetWebURLRequestHeaders(info.urlRequest), referrer,
+ GetRequestBodyForWebURLRequest(info.urlRequest), referrer,
info.defaultPolicy, info.replacesCurrentHistoryItem, true);
// Suppress the load in Blink but mark the frame as loading.
return blink::WebNavigationPolicyHandledByClient;
@@ -5177,7 +5174,6 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
if (should_fork) {
OpenURL(url, IsHttpPost(info.urlRequest),
GetRequestBodyForWebURLRequest(info.urlRequest),
- GetWebURLRequestHeaders(info.urlRequest),
send_referrer ? referrer : Referrer(), info.defaultPolicy,
info.replacesCurrentHistoryItem, false);
return blink::WebNavigationPolicyIgnore; // Suppress the load here.
@@ -5219,8 +5215,7 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
if (is_fork) {
// Open the URL via the browser, not via WebKit.
OpenURL(url, IsHttpPost(info.urlRequest),
- GetRequestBodyForWebURLRequest(info.urlRequest),
- GetWebURLRequestHeaders(info.urlRequest), Referrer(),
+ GetRequestBodyForWebURLRequest(info.urlRequest), Referrer(),
info.defaultPolicy, info.replacesCurrentHistoryItem, false);
return blink::WebNavigationPolicyIgnore;
}
@@ -5557,7 +5552,6 @@ void RenderFrameImpl::OpenURL(
const GURL& url,
bool uses_post,
const scoped_refptr<ResourceRequestBodyImpl>& resource_request_body,
- const std::string& extra_headers,
const Referrer& referrer,
WebNavigationPolicy policy,
bool should_replace_current_entry,
@@ -5566,7 +5560,6 @@ void RenderFrameImpl::OpenURL(
params.url = url;
params.uses_post = uses_post;
params.resource_request_body = resource_request_body;
- params.extra_headers = extra_headers;
params.referrer = referrer;
params.disposition = RenderViewImpl::NavigationPolicyToDisposition(policy);
@@ -6058,7 +6051,8 @@ void RenderFrameImpl::BeginNavigation(const NavigationPolicyInfo& info) {
REQUEST_CONTEXT_FRAME_TYPE_NESTED);
Send(new FrameHostMsg_BeginNavigation(
- routing_id_, MakeCommonNavigationParams(info),
+ routing_id_,
+ MakeCommonNavigationParams(info),
BeginNavigationParams(
GetWebURLRequestHeaders(info.urlRequest),
GetLoadFlagsForWebURLRequest(info.urlRequest),
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | content/renderer/render_frame_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698