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

Unified Diff: chrome/browser/sync/sync_ui_util.cc

Issue 479353003: Add IsAuthenticated() method to SigninManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments Created 6 years, 4 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: chrome/browser/sync/sync_ui_util.cc
diff --git a/chrome/browser/sync/sync_ui_util.cc b/chrome/browser/sync/sync_ui_util.cc
index d67b68dee44ebd5b8bee0bcce933cc0a6aae1523..3cc618942af8f87a1b17a0c101ac4584d46bd842 100644
--- a/chrome/browser/sync/sync_ui_util.cc
+++ b/chrome/browser/sync/sync_ui_util.cc
@@ -140,7 +140,7 @@ MessageType GetStatusInfo(ProfileSyncService* service,
MessageType result_type(SYNCED);
- if (signin.GetAuthenticatedUsername().empty())
+ if (!signin.IsAuthenticated())
return PRE_SYNCED;
if (!service || service->IsManaged() || service->HasSyncSetupCompleted() ||
@@ -265,7 +265,7 @@ MessageType GetStatusInfo(ProfileSyncService* service,
} else if (status_label) {
status_label->assign(l10n_util::GetStringUTF16(IDS_SYNC_SETUP_ERROR));
}
- } else if (!signin.GetAuthenticatedUsername().empty()) {
+ } else if (signin.IsAuthenticated()) {
// The user is signed in, but sync has been stopped.
if (status_label) {
base::string16 label = l10n_util::GetStringFUTF16(

Powered by Google App Engine
This is Rietveld 408576698