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

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

Issue 22904002: When switching processes during redirects, update the cookie policy (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates Created 7 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_loader.cc
diff --git a/content/browser/loader/resource_loader.cc b/content/browser/loader/resource_loader.cc
index b43d5d627912dca5c3a702ecc3fae58ed44cfb5b..5b0a9f4a284fe0997a4486f6c9ea7e6293fba2c8 100644
--- a/content/browser/loader/resource_loader.cc
+++ b/content/browser/loader/resource_loader.cc
@@ -155,9 +155,15 @@ void ResourceLoader::ReportUploadProgress() {
}
}
-void ResourceLoader::MarkAsTransferring() {
+void ResourceLoader::MarkAsTransferring(const GURL& target_url) {
+ CHECK_EQ(GetRequestInfo()->GetResourceType(), ResourceType::MAIN_FRAME)
+ << "Cannot transfer non-main frame navigations";
is_transferring_ = true;
+ // When transferring a request to another process, the renderer doesn't get
+ // a chance to update the cookie policy URL. Do it here instead.
+ request()->set_first_party_for_cookies(target_url);
+
// When an URLRequest is transferred to a new RenderViewHost, its
// ResourceHandler should not receive any notifications because it may depend
// on the state of the old RVH. We set a ResourceHandler that only allows
« no previous file with comments | « content/browser/loader/resource_loader.h ('k') | content/browser/loader/transfer_navigation_resource_throttle.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698