IdentityManager: Remove is_valid from OnRefreshTokenUpdatedForAccount
|is_valid| is confusing for people in practice: many people want to
check is_valid in cases where they shouldn't. I introduced it to avoid
adding the ability to query the persistent error state to the
IdentityManager API (for Sync's use case of detecting the refresh token
being set to the invalid token). However, avoiding adding that API is no
longer a priority.
This CL gets rid of this parameter altogether; after precedent CLs, it
is not used by any consumer, so this is just stripping out the param.
TBR=sdefresne@chromium.org
Bug: 908412
Change-Id: Idfa954b8d827312c88f2a93dab0e76a87e9ba147
Reviewed-on: https://chromium-review.googlesource.com/c/1350986
Reviewed-by: Colin Blundell <blundell@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#612650}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: bc1d0fc31510d914bb085f7c8248991b2669509c2 files changed
