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

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

Issue 2349823003: Pass a RemovalCause to CookieChangedCallback (Closed)
Patch Set: rebase Created 4 years, 3 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_cookie_changed_subscription.cc
diff --git a/components/signin/core/browser/signin_cookie_changed_subscription.cc b/components/signin/core/browser/signin_cookie_changed_subscription.cc
index bfcfde49dcf5180d7d1c0e371ac19df324137a3e..3f4325259372fb1b11e1324bfc9bcc428a7398e8 100644
--- a/components/signin/core/browser/signin_cookie_changed_subscription.cc
+++ b/components/signin/core/browser/signin_cookie_changed_subscription.cc
@@ -87,19 +87,17 @@ void SigninCookieChangedSubscription::RunAsyncOnCookieChanged(
scoped_refptr<base::TaskRunner> proxy,
base::WeakPtr<SigninCookieChangedSubscription> subscription,
const net::CanonicalCookie& cookie,
- bool removed) {
+ net::CookieStore::ChangeCause cause) {
proxy->PostTask(FROM_HERE,
base::Bind(&SigninCookieChangedSubscription::OnCookieChanged,
- subscription,
- cookie,
- removed));
+ subscription, cookie, cause));
}
void SigninCookieChangedSubscription::OnCookieChanged(
const net::CanonicalCookie& cookie,
- bool removed) {
+ net::CookieStore::ChangeCause cause) {
DCHECK(thread_checker_.CalledOnValidThread());
if (!callback_.is_null()) {
- callback_.Run(cookie, removed);
+ callback_.Run(cookie, cause);
}
}
« no previous file with comments | « components/signin/core/browser/signin_cookie_changed_subscription.h ('k') | ios/net/cookies/cookie_store_ios.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698