• James Cook's avatar
    Migrate SigninErrorNotifier to unconsented primary account · 0e2fa9d7
    James Cook authored
    SplitSettingsSync will allow Chrome OS users to opt-out of browser
    sync.  However, IdentityAccessor::GetPrimaryAccountInfo() assumes
    the user has consented to browser sync. We still need to detect cases
    where the refresh token is invalid, even if the user has not consented
    to browser sync.
    
    Switch to using the "unconsented" primary account. On Chrome OS this
    account always exists for the logged-in user account, whether or not
    the user consented to browser sync.
    
    See go/cros-sync-mock and go/cros-primary-account for details.
    
    Bug: 1042400, 760610
    Test: updated existing components_unittests
    Test: Bug 760610 still does not repro with SplitSettingsSync enabled
    Change-Id: I030e45eca6bcf82381bf1be9a4e6f829b7a4a3a1
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2040531
    Commit-Queue: David Roger <droger@chromium.org>
    Reviewed-by: default avatarDavid Roger <droger@chromium.org>
    Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Auto-Submit: James Cook <jamescook@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#739314}
    0e2fa9d7
auth_sync_observer.cc 4.21 KB