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

Unified Diff: net/base/network_delegate.h

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/base/network_delegate.h
diff --git a/net/base/network_delegate.h b/net/base/network_delegate.h
index cad221e99db28d8395ad3f35ccfeb196d8e1db55..5bf64313a1bc6604ed2a948b01982e2581c0fd3b 100644
--- a/net/base/network_delegate.h
+++ b/net/base/network_delegate.h
@@ -109,8 +109,9 @@ class NET_EXPORT NetworkDelegate : public base::NonThreadSafe {
const GURL& first_party_for_cookies) const;
// TODO(mkwst): Remove this once we decide whether or not we wish to ship
- // first-party cookies. https://crbug.com/459154
- bool FirstPartyOnlyCookieExperimentEnabled() const;
+ // first-party cookies and cookie prefixes. https://crbug.com/459154,
+ // https://crbug.com/541511
+ bool ExperimentalCookieFeaturesEnabled() const;
bool CancelURLRequestWithPolicyViolatingReferrerHeader(
const URLRequest& request,
@@ -294,12 +295,13 @@ class NET_EXPORT NetworkDelegate : public base::NonThreadSafe {
const GURL& url,
const GURL& first_party_for_cookies) const = 0;
- // Returns true if the embedder has enabled the "first-party" cookie
- // experiment, and false otherwise.
+ // Returns true if the embedder has enabled the experimental features,
+ // and false otherwise.
//
// TODO(mkwst): Remove this once we decide whether or not we wish to ship
- // first-party cookies. https://crbug.com/459154
- virtual bool OnFirstPartyOnlyCookieExperimentEnabled() const = 0;
+ // first-party cookies and cookie prefixes. https://crbug.com/459154,
+ // https://crbug.com/541511
+ virtual bool OnExperimentalCookieFeaturesEnabled() const = 0;
mmenke 2015/10/12 14:25:42 Maybe OnAreExperimental...? This sounds like it's
estark 2015/10/12 14:48:08 Done.
// Called when the |referrer_url| for requesting |target_url| during handling
// of the |request| is does not comply with the referrer policy (e.g. a

Powered by Google App Engine
This is Rietveld 408576698