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

Side by Side Diff: components/signin/core/browser/signin_tracker.cc

Issue 1075273002: Handle ListAccount fetches from within the GaiaCookieManagerService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix a test post rebase Created 5 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/signin/core/browser/signin_tracker.h" 5 #include "components/signin/core/browser/signin_tracker.h"
6 6
7 #include "components/signin/core/browser/account_reconcilor.h"
8 #include "components/signin/core/browser/gaia_cookie_manager_service.h" 7 #include "components/signin/core/browser/gaia_cookie_manager_service.h"
9 #include "components/signin/core/browser/profile_oauth2_token_service.h" 8 #include "components/signin/core/browser/profile_oauth2_token_service.h"
10 #include "components/signin/core/browser/signin_client.h" 9 #include "components/signin/core/browser/signin_client.h"
11 #include "google_apis/gaia/gaia_constants.h" 10 #include "google_apis/gaia/gaia_constants.h"
12 11
13 SigninTracker::SigninTracker(ProfileOAuth2TokenService* token_service, 12 SigninTracker::SigninTracker(ProfileOAuth2TokenService* token_service,
14 SigninManagerBase* signin_manager, 13 SigninManagerBase* signin_manager,
15 AccountReconcilor* account_reconcilor,
16 GaiaCookieManagerService* cookie_manager_service, 14 GaiaCookieManagerService* cookie_manager_service,
17 SigninClient* client, 15 SigninClient* client,
18 Observer* observer) 16 Observer* observer)
19 : token_service_(token_service), 17 : token_service_(token_service),
20 signin_manager_(signin_manager), 18 signin_manager_(signin_manager),
21 account_reconcilor_(account_reconcilor),
22 cookie_manager_service_(cookie_manager_service), 19 cookie_manager_service_(cookie_manager_service),
23 client_(client), 20 client_(client),
24 observer_(observer) { 21 observer_(observer) {
25 Initialize(); 22 Initialize();
26 } 23 }
27 24
28 SigninTracker::~SigninTracker() { 25 SigninTracker::~SigninTracker() {
29 signin_manager_->RemoveObserver(this); 26 signin_manager_->RemoveObserver(this);
30 token_service_->RemoveObserver(this); 27 token_service_->RemoveObserver(this);
31 cookie_manager_service_->RemoveObserver(this); 28 cookie_manager_service_->RemoveObserver(this);
(...skipping 19 matching lines...) Expand all
51 48
52 void SigninTracker::OnRefreshTokenRevoked(const std::string& account_id) { 49 void SigninTracker::OnRefreshTokenRevoked(const std::string& account_id) {
53 NOTREACHED(); 50 NOTREACHED();
54 } 51 }
55 52
56 void SigninTracker::OnAddAccountToCookieCompleted( 53 void SigninTracker::OnAddAccountToCookieCompleted(
57 const std::string& account_id, 54 const std::string& account_id,
58 const GoogleServiceAuthError& error) { 55 const GoogleServiceAuthError& error) {
59 observer_->MergeSessionComplete(error); 56 observer_->MergeSessionComplete(error);
60 } 57 }
OLDNEW
« no previous file with comments | « components/signin/core/browser/signin_tracker.h ('k') | components/signin/ios/browser/merge_session_observer_bridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698