Commit b614084e authored by Jérôme Lebel's avatar Jérôme Lebel Committed by Commit Bot

[iOS] Fixing account reconciler

The primary identity is now set after loading the accounts. So the
reconciler needs to start once the primary identity is set.

This patch fixes those failing tests:
-[ExternalURLSigninTestCase testSignInAddAccountCommand]
-[ExternalURLSigninTestCase testSignInReauthenticate]
-[ExternalURLSigninTestCase testSignInOneUser]

Bug: 983141
Change-Id: Iaa6c47b64573838b55035d3b78dcb50dc2901d3a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1698506
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Auto-Submit: Jérôme Lebel <jlebel@chromium.org>
Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#676857}
parent fddb08d9
...@@ -250,11 +250,8 @@ void AccountReconcilor::SetIsWKHTTPSystemCookieStoreEnabled(bool is_enabled) { ...@@ -250,11 +250,8 @@ void AccountReconcilor::SetIsWKHTTPSystemCookieStoreEnabled(bool is_enabled) {
void AccountReconcilor::EnableReconcile() { void AccountReconcilor::EnableReconcile() {
SetState(AccountReconcilorState::ACCOUNT_RECONCILOR_SCHEDULED); SetState(AccountReconcilorState::ACCOUNT_RECONCILOR_SCHEDULED);
RegisterWithAllDependencies(); RegisterWithAllDependencies();
#if !defined(OS_IOS)
// TODO(droger): Investigate why this breaks tests on iOS.
if (IsIdentityManagerReady()) if (IsIdentityManagerReady())
StartReconcile(); StartReconcile();
#endif // !defined(OS_IOS)
} }
void AccountReconcilor::DisableReconcile(bool logout_all_accounts) { void AccountReconcilor::DisableReconcile(bool logout_all_accounts) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment