Sync: Replace outdated "authenticated / primary account" terminology
This CL rename SyncService::IsAuthenticatedAccountPrimary() with
HasSyncConsent(), and GetAuthenticatedAccountInfo() with just
GetAccountInfo().
"Authenticated account" is some outdated terminology that's not used
in sign-in code anymore.
More importantly, "primary account" was used here in an outdated
meaning that wasn't consistent anymore with how it's used in signin
code.
This is a pure rename; no behavior changes.
Bug: 1254280
Change-Id: Ice60b4d790d4fc7451c530705a73ac87ba4606f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3217741
Reviewed-by: Colin Blundell <[email protected]>
Reviewed-by: Sylvain Defresne <[email protected]>
Reviewed-by: Mihai Sardarescu <[email protected]>
Commit-Queue: Marc Treib <[email protected]>
Cr-Commit-Position: refs/heads/main@{#932995}
diff --git a/components/sync/driver/sync_service.cc b/components/sync/driver/sync_service.cc
index 35c4383..dbf3e8e 100644
--- a/components/sync/driver/sync_service.cc
+++ b/components/sync/driver/sync_service.cc
@@ -34,7 +34,7 @@
}
bool SyncService::CanSyncFeatureStart() const {
- return GetDisableReasons().Empty() && IsAuthenticatedAccountPrimary();
+ return GetDisableReasons().Empty() && HasSyncConsent();
}
bool SyncService::IsEngineInitialized() const {
