Chromium Code Reviews| Index: chrome/browser/signin/chrome_signin_client.h |
| diff --git a/chrome/browser/signin/chrome_signin_client.h b/chrome/browser/signin/chrome_signin_client.h |
| index 98d1021273238492bbcd91d0df09cae3f4304987..76afa02b3bf86b1f4770e77bad96600139ec7bbd 100644 |
| --- a/chrome/browser/signin/chrome_signin_client.h |
| +++ b/chrome/browser/signin/chrome_signin_client.h |
| @@ -56,8 +56,8 @@ class ChromeSigninClient : public SigninClient, |
| // <Build Info> <OS> <Version number> (<Last change>)<channel or "-devel"> |
| // If version information is unavailable, returns "invalid." |
| virtual std::string GetProductVersion() OVERRIDE; |
| - virtual void SetCookieChangedCallback(const CookieChangedCallback& callback) |
| - OVERRIDE; |
| + virtual scoped_ptr<CookieChangedCallbackList::Subscription> |
| + AddCookieChangedCallback(const CookieChangedCallback& callback) OVERRIDE; |
| virtual void GoogleSigninSucceeded(const std::string& username, |
| const std::string& password) OVERRIDE; |
| @@ -73,9 +73,9 @@ class ChromeSigninClient : public SigninClient, |
| Profile* profile_; |
| content::NotificationRegistrar registrar_; |
| - // The callback that if non-empty will be called when notifications about |
| - // cookie changes are received. |
| - CookieChangedCallback callback_; |
| + // The callbacks that will be called when notifications about cookie changes |
| + // are received. |
| + base::CallbackList<void(const net::CanonicalCookie* cookie)> callbacks_; |
|
Roger Tawa OOO till Jul 10th
2014/08/06 20:29:42
Should this be: base::CallbackList<CookieChangedC
Mike Lerman
2014/08/07 16:39:26
The type for the CallbackList has to be the functi
|
| // See SetSigninProcess. Tracks the currently active signin process |
| // by ID, if there is one. |