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

Unified Diff: components/signin/core/browser/signin_header_helper.cc

Issue 2655443003: Unify the "get" and "set" cookie access settings. (Closed)
Patch Set: fix android Created 3 years, 11 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: components/signin/core/browser/signin_header_helper.cc
diff --git a/components/signin/core/browser/signin_header_helper.cc b/components/signin/core/browser/signin_header_helper.cc
index 0d192ea566867a0307b7297ea0bcb8ff1d482f4f..668d25d449220e5f1700d141191a8fd0cfa2e362 100644
--- a/components/signin/core/browser/signin_header_helper.cc
+++ b/components/signin/core/browser/signin_header_helper.cc
@@ -164,8 +164,8 @@ bool SettingsAllowSigninCookies(
GURL gaia_url = GaiaUrls::GetInstance()->gaia_url();
GURL google_url = GaiaUrls::GetInstance()->google_url();
return cookie_settings &&
- cookie_settings->IsSettingCookieAllowed(gaia_url, gaia_url) &&
- cookie_settings->IsSettingCookieAllowed(google_url, google_url);
+ cookie_settings->IsCookieAccessAllowed(gaia_url, gaia_url) &&
+ cookie_settings->IsCookieAccessAllowed(google_url, google_url);
}
std::string BuildMirrorRequestCookieIfPossible(

Powered by Google App Engine
This is Rietveld 408576698