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

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

Issue 1339783003: Revert of Add logging to profile downloader to diagnose Android FR issue. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/profiles/profile_downloader.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/account_fetcher_service.h" 5 #include "components/signin/core/browser/account_fetcher_service.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/metrics/field_trial.h" 8 #include "base/metrics/field_trial.h"
9 #include "base/prefs/pref_service.h" 9 #include "base/prefs/pref_service.h"
10 #include "base/profiler/scoped_tracker.h" 10 #include "base/profiler/scoped_tracker.h"
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 // Now that network fetches are enabled, schedule the next refresh. 112 // Now that network fetches are enabled, schedule the next refresh.
113 ScheduleNextRefresh(); 113 ScheduleNextRefresh();
114 } 114 }
115 115
116 bool AccountFetcherService::IsAllUserInfoFetched() const { 116 bool AccountFetcherService::IsAllUserInfoFetched() const {
117 return user_info_requests_.empty(); 117 return user_info_requests_.empty();
118 } 118 }
119 119
120 void AccountFetcherService::FetchUserInfoBeforeSignin( 120 void AccountFetcherService::FetchUserInfoBeforeSignin(
121 const std::string& account_id) { 121 const std::string& account_id) {
122 LOG(WARNING) << "Fetching account info before signin for account id: "
123 << account_id;
124 RefreshAccountInfo(account_id, false); 122 RefreshAccountInfo(account_id, false);
125 } 123 }
126 124
127 void AccountFetcherService::RefreshAllAccountInfo(bool only_fetch_if_invalid) { 125 void AccountFetcherService::RefreshAllAccountInfo(bool only_fetch_if_invalid) {
128 std::vector<std::string> accounts = token_service_->GetAccounts(); 126 std::vector<std::string> accounts = token_service_->GetAccounts();
129 for (std::vector<std::string>::const_iterator it = accounts.begin(); 127 for (std::vector<std::string>::const_iterator it = accounts.begin();
130 it != accounts.end(); ++it) { 128 it != accounts.end(); ++it) {
131 RefreshAccountInfo(*it, only_fetch_if_invalid); 129 RefreshAccountInfo(*it, only_fetch_if_invalid);
132 } 130 }
133 } 131 }
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
315 UpdateChildInfo(); 313 UpdateChildInfo();
316 account_tracker_service_->StopTrackingAccount(account_id); 314 account_tracker_service_->StopTrackingAccount(account_id);
317 } 315 }
318 316
319 void AccountFetcherService::OnRefreshTokensLoaded() { 317 void AccountFetcherService::OnRefreshTokensLoaded() {
320 // OnRefreshTokenAvailable has been called for all accounts by this point. 318 // OnRefreshTokenAvailable has been called for all accounts by this point.
321 // Maybe remove this after further investigation. 319 // Maybe remove this after further investigation.
322 RefreshAllAccountInfo(true); 320 RefreshAllAccountInfo(true);
323 UpdateChildInfo(); 321 UpdateChildInfo();
324 } 322 }
OLDNEW
« no previous file with comments | « chrome/browser/profiles/profile_downloader.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698