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

Unified Diff: net/url_request/url_request_http_job.cc

Issue 1392853006: Rename NetworkDelegate first-party-only flag (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more renaming Created 5 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
Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index c317469cb137ad7ec840f229a7ffeb26dce62ad1..378f6f942d923e7ec4b65f2ad03d3633ddf9e4fb 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -661,7 +661,7 @@ void URLRequestHttpJob::DoLoadCookies() {
// TODO(mkwst): Drop this `if` once we decide whether or not to ship
// first-party cookies: https://crbug.com/459154
if (network_delegate() &&
- network_delegate()->FirstPartyOnlyCookieExperimentEnabled())
+ network_delegate()->ExperimentalCookieFeaturesEnabled())
mmenke 2015/10/12 14:25:42 Should be calling the On* method here (Old bug...)
estark 2015/10/12 14:48:08 How so? The On* version is private.
options.set_first_party_url(request_->first_party_for_cookies());
else
options.set_include_first_party_only();

Powered by Google App Engine
This is Rietveld 408576698