Issue metadata
Sign in to add a comment
|
Convert chrome/browser/ui/webui/settings/people_handler.cc to IdentityManager |
||||||||||||||||||||||
Issue descriptionAPI used in people_handler.cc: - SigninManager::SignOut() - SigninManager::IsAuthenticated() - SigninManager::SignOutAndRemoveAllAccounts() - SigninManager::IsSigninAllowed()
,
Jan 16
(6 days ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cacfd35f4e09cbbd969a2c882f3db0eccc127bc0 commit cacfd35f4e09cbbd969a2c882f3db0eccc127bc0 Author: Mario Sanchez Prada <mario@igalia.com> Date: Wed Jan 16 10:36:33 2019 Migrate PeopleHandler to the IdentityManager Use the APIs from IdentityManager and PrimaryAccountsMutator to migrate calls previously depending on SigninManager and ProfileOAuth2TokenService. Bug: 898810 , 920236 , 920237 , 921445 Change-Id: I22b046bae82e2539d165d95573c27d47affa6a06 Reviewed-on: https://chromium-review.googlesource.com/c/1298994 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#623185} [modify] https://crrev.com/cacfd35f4e09cbbd969a2c882f3db0eccc127bc0/chrome/browser/ui/webui/settings/people_handler.cc [modify] https://crrev.com/cacfd35f4e09cbbd969a2c882f3db0eccc127bc0/chrome/browser/ui/webui/settings/people_handler_unittest.cc |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by gyuyo...@igalia.com
, Jan 14Status: Duplicate (was: Started)