New issue
Advanced search Search tips

Issue 910196 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 908858



Sign in to add a comment

Convert CWVSyncController to use IdentityManager

Project Member Reported by sdefresne@chromium.org, Nov 29

Issue description

This will require changing the class to receive an IdentityManager instead of SigninManager, ProfileOAuth2TokenService and AccountTrackerService.

Methods with replacements:
- SigninManager::GetAuthenticatedAccountId
  => IdentityManager::GetPrimaryAccountId
- SigninManager::OnExternalSigninCompleted
  => PrimaryAccountMutator::SetPrimaryAccount
- SigninManager::SignOut
  => PrimaryAccountMutator::ClearPrimaryAccount
- AccountTrackerService::GetAccountInfo
  => IdentityManager::GetAccountInfoForAccountWithRefreshToken

Methods with no replacements:
- AccountTrackerService::SeedAccountInfo
- ProfileOAuth2TokenServiceIOSDelegate::ReloadCredentials

 
Note: The conversions from PO2TS and SigninManager are tracked as P1 bugs in crbug.com/913927 and crbug.com/903838 respectively.

Sign in to add a comment