IdentityManager: Have OnRefreshTokenRemovedForAccount() pass account ID
We will shortly be changing IdentityManager to fire
IdentityManager::Observer::OnRefreshTokenRemovedForAccount() as a
straight passthrough from
ProfileOAuth2TokenService::Observer::OnRefreshTokenRevoked() as part of
crbug.com/883722. However, this raises a challenge in that the
AccountInfo isn't guaranteed to be in AccountTrackerService when this
callback is received by IdentityManager.
To prepare for this change, this CL changes
IdentityManager::Observer::OnRefreshTokenRemovedForAccount() to pass
the account ID. No observer requires anything else from the AccountInfo
that is currently being passed.
Bug: 883722
Change-Id: Iccd921a11ab229b897028987e538cc799ed35026
Reviewed-on: https://chromium-review.googlesource.com/1225980
Reviewed-by: Sylvain Defresne <[email protected]>
Commit-Queue: Colin Blundell <[email protected]>
Cr-Commit-Position: refs/heads/master@{#592726}diff --git a/components/invalidation/impl/profile_identity_provider.cc b/components/invalidation/impl/profile_identity_provider.cc
index 09f3d33..34247778 100644
--- a/components/invalidation/impl/profile_identity_provider.cc
+++ b/components/invalidation/impl/profile_identity_provider.cc
@@ -116,8 +116,8 @@
}
void ProfileIdentityProvider::OnRefreshTokenRemovedForAccount(
- const AccountInfo& account_info) {
- ProcessRefreshTokenRemovalForAccount(account_info.account_id);
+ const std::string& account_id) {
+ ProcessRefreshTokenRemovalForAccount(account_id);
}
} // namespace invalidation
