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

Unified Diff: content/browser/loader/resource_dispatcher_host_impl.cc

Issue 1271733002: [2/3 chromium] Support redirect option of Request and "opaqueredirect" response type. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: check deffered Created 5 years, 4 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/loader/resource_dispatcher_host_impl.cc
diff --git a/content/browser/loader/resource_dispatcher_host_impl.cc b/content/browser/loader/resource_dispatcher_host_impl.cc
index 5600a23455fc80f8f2a6592077a78be2babcb165..f21be7de1d80a39e17cb33f997387dec6deba04a 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.cc
+++ b/content/browser/loader/resource_dispatcher_host_impl.cc
@@ -1281,6 +1281,7 @@ void ResourceDispatcherHostImpl::BeginRequest(
request_data.resource_type,
request_data.transition_type,
request_data.should_replace_current_entry,
+ request_data.fetch_redirect_mode,
false, // is download
false, // is stream
allow_download,
@@ -1307,17 +1308,12 @@ void ResourceDispatcherHostImpl::BeginRequest(
// Initialize the service worker handler for the request. We don't use
// ServiceWorker for synchronous loads to avoid renderer deadlocks.
ServiceWorkerRequestHandler::InitializeHandler(
- new_request.get(),
- filter_->service_worker_context(),
- blob_context,
- child_id,
- request_data.service_worker_provider_id,
+ new_request.get(), filter_->service_worker_context(), blob_context,
+ child_id, request_data.service_worker_provider_id,
request_data.skip_service_worker || is_sync_load,
- request_data.fetch_request_mode,
- request_data.fetch_credentials_mode,
- request_data.resource_type,
- request_data.fetch_request_context_type,
- request_data.fetch_frame_type,
+ request_data.fetch_request_mode, request_data.fetch_credentials_mode,
+ request_data.fetch_redirect_mode, request_data.resource_type,
+ request_data.fetch_request_context_type, request_data.fetch_frame_type,
request_data.request_body);
// Have the appcache associate its extra info with the request.
@@ -1540,6 +1536,7 @@ ResourceRequestInfoImpl* ResourceDispatcherHostImpl::CreateRequestInfo(
RESOURCE_TYPE_SUB_RESOURCE,
ui::PAGE_TRANSITION_LINK,
false, // should_replace_current_entry
+ FETCH_REDIRECT_MODE_FOLLOW,
download, // is_download
false, // is_stream
download, // allow_download
@@ -1980,6 +1977,7 @@ void ResourceDispatcherHostImpl::BeginNavigationRequest(
// should_replace_current_entry. This was only maintained at layer for
// request transfers and isn't needed for browser-side navigations.
false,
+ FETCH_REDIRECT_MODE_FOLLOW,
false, // is download
false, // is stream
info.common_params.allow_download,

Powered by Google App Engine
This is Rietveld 408576698