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

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

Issue 1002363002: Remove prerender cookie store, part 3. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@prerender-revert-cookie-store-2
Patch Set: also remove hook from ContentBrowserClient Created 5 years, 9 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 | « no previous file | content/browser/renderer_host/render_message_filter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 61e458d5a7c4d8a7165d50b85d15f440c4dc6674..53bd1e4f2f423f8e458f01afc69535c1b98a8cce 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.cc
+++ b/content/browser/loader/resource_dispatcher_host_impl.cc
@@ -1153,12 +1153,9 @@ void ResourceDispatcherHostImpl::BeginRequest(
}
// Construct the request.
- net::CookieStore* cookie_store =
- GetContentClient()->browser()->OverrideCookieStoreForRenderProcess(
- child_id);
scoped_ptr<net::URLRequest> new_request;
new_request = request_context->CreateRequest(
- request_data.url, request_data.priority, NULL, cookie_store);
+ request_data.url, request_data.priority, NULL, NULL);
new_request->set_method(request_data.method);
new_request->set_first_party_for_cookies(
@@ -1594,13 +1591,8 @@ void ResourceDispatcherHostImpl::BeginSaveFile(
return;
}
- net::CookieStore* cookie_store =
- GetContentClient()->browser()->OverrideCookieStoreForRenderProcess(
- child_id);
scoped_ptr<net::URLRequest> request(
- request_context->CreateRequest(url, net::DEFAULT_PRIORITY, NULL,
- cookie_store));
-
+ request_context->CreateRequest(url, net::DEFAULT_PRIORITY, NULL, NULL));
request->set_method("GET");
SetReferrerForRequest(request.get(), referrer);
@@ -1903,9 +1895,6 @@ void ResourceDispatcherHostImpl::BeginNavigationRequest(
// requests that have the ignore limits flag set.
DCHECK(!(load_flags & net::LOAD_IGNORE_LIMITS));
- // TODO(davidben): OverrideCookieStoreForRenderProcess handling for
- // prerender. There may not be a renderer process yet, so we need to use the
- // ResourceContext or something.
scoped_ptr<net::URLRequest> new_request;
new_request = request_context->CreateRequest(
info.common_params.url, net::HIGHEST, nullptr, nullptr);
« no previous file with comments | « no previous file | content/browser/renderer_host/render_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698