Add missing API to replace AccountTrackerService::Observer::OnAccountUpdated |
|||||||||||||||||||
Issue description
The API would probably be the following:
void IdentityManager::Observer::OnAccountUpdated(
const AccountInfo& account_info);
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(6 days ago)
,
Jan 17
(5 days ago)
,
Jan 18
(4 days ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b1d45207353420b1fbcad9f0dd426d65ad5e9255 commit b1d45207353420b1fbcad9f0dd426d65ad5e9255 Author: Julie Jeongeun Kim <jkim@igalia.com> Date: Fri Jan 18 15:00:57 2019 [s13n] Add IdentityManager::Observer::OnAccountUpdated This CL adds IdentityManager::Observer::OnAccountUpdated to replace AccountTrackerService::Observer::OnAccountUpdated. Test: IdentityManagerTest.ObserveOnAccountUpdated Bug: 922469 Change-Id: I2e787dca982dbb935925f6a00dd239aa3fe915a8 Reviewed-on: https://chromium-review.googlesource.com/c/1417433 Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#624123} [modify] https://crrev.com/b1d45207353420b1fbcad9f0dd426d65ad5e9255/services/identity/public/cpp/identity_manager.cc [modify] https://crrev.com/b1d45207353420b1fbcad9f0dd426d65ad5e9255/services/identity/public/cpp/identity_manager.h [modify] https://crrev.com/b1d45207353420b1fbcad9f0dd426d65ad5e9255/services/identity/public/cpp/identity_manager_unittest.cc
,
Yesterday
(30 hours ago)
|
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by sdefresne@chromium.org
, Jan 16 (6 days ago)