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

[iOS] Fixing AddAccountSigninCoordinator when being interrupted

When the coordinator is interrupted, the ChromeIdentityInteractionManager
needs to be dismissed. Once it is dismissed, the manager can be set to
nil.

Bug: 971989
Change-Id: I0da58b71e3e151d71e76fce5d9a8f53c00e66b72
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2154185Reviewed-by: default avatarNohemi Fernandez <fernandex@chromium.org>
Commit-Queue: Nohemi Fernandez <fernandex@chromium.org>
Cr-Commit-Position: refs/heads/master@{#759979}
parent 44ec5793
...@@ -95,6 +95,8 @@ using signin_metrics::PromoAction; ...@@ -95,6 +95,8 @@ using signin_metrics::PromoAction;
[self.identityInteractionManager cancelAndDismissAnimated:NO]; [self.identityInteractionManager cancelAndDismissAnimated:NO];
break; break;
} }
// The manager UI has been dismissed, it can be set to nil.
self.identityInteractionManager = nil;
[self runCompletionCallbackWithSigninResult:SigninCoordinatorResultInterrupted [self runCompletionCallbackWithSigninResult:SigninCoordinatorResultInterrupted
identity:nil identity:nil
showAdvancedSettingsSignin:NO]; showAdvancedSettingsSignin:NO];
......
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