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

Side by Side Diff: components/signin/core/browser/account_reconcilor.h

Issue 896723006: Avoid double registration with content settings in account reconcilor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « no previous file | components/signin/core/browser/account_reconcilor.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_ 4 #ifndef COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_
5 #define COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_ 5 #define COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_
6 6
7 #include <deque> 7 #include <deque>
8 #include <functional> 8 #include <functional>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 // The SigninManager associated with this reconcilor. 186 // The SigninManager associated with this reconcilor.
187 SigninManagerBase* signin_manager_; 187 SigninManagerBase* signin_manager_;
188 188
189 // The SigninClient associated with this reconcilor. 189 // The SigninClient associated with this reconcilor.
190 SigninClient* client_; 190 SigninClient* client_;
191 191
192 MergeSessionHelper merge_session_helper_; 192 MergeSessionHelper merge_session_helper_;
193 scoped_ptr<GaiaAuthFetcher> gaia_fetcher_; 193 scoped_ptr<GaiaAuthFetcher> gaia_fetcher_;
194 bool registered_with_token_service_; 194 bool registered_with_token_service_;
195 bool registered_with_merge_session_helper_; 195 bool registered_with_merge_session_helper_;
196 bool registered_with_content_settings_;
196 197
197 // True while the reconcilor is busy checking or managing the accounts in 198 // True while the reconcilor is busy checking or managing the accounts in
198 // this profile. 199 // this profile.
199 bool is_reconcile_started_; 200 bool is_reconcile_started_;
200 base::Time m_reconcile_start_time_; 201 base::Time m_reconcile_start_time_;
201 202
202 // True iff this is the first time the reconcilor is executing. 203 // True iff this is the first time the reconcilor is executing.
203 bool first_execution_; 204 bool first_execution_;
204 205
205 // Used during reconcile action. 206 // Used during reconcile action.
(...skipping 14 matching lines...) Expand all
220 221
221 std::deque<GetAccountsFromCookieCallback> get_gaia_accounts_callbacks_; 222 std::deque<GetAccountsFromCookieCallback> get_gaia_accounts_callbacks_;
222 223
223 scoped_ptr<SigninClient::CookieChangedSubscription> 224 scoped_ptr<SigninClient::CookieChangedSubscription>
224 cookie_changed_subscription_; 225 cookie_changed_subscription_;
225 226
226 DISALLOW_COPY_AND_ASSIGN(AccountReconcilor); 227 DISALLOW_COPY_AND_ASSIGN(AccountReconcilor);
227 }; 228 };
228 229
229 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_ 230 #endif // COMPONENTS_SIGNIN_CORE_BROWSER_ACCOUNT_RECONCILOR_H_
OLDNEW
« no previous file with comments | « no previous file | components/signin/core/browser/account_reconcilor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698