commit | 575d2274359529d2010b14fef42981ec99ad5777 | [log] [tgz] |
---|---|---|
author | Colin Blundell <blundell@chromium.org> | Mon Sep 24 15:32:21 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Mon Sep 24 15:32:21 2018 |
tree | cdd196178a233afa1fe86d1297564eb2b76b3d1e | |
parent | 6bda5adaeec226b3e2745fd0426098190b46241b [diff] |
OAuth2TokenService.java: Remove unused methods and their tests OAuth2TokenService.java has methods that trigger the firing of various refresh token-related notifications by ProfileOAuth2TokenService [the C++ class]. These methods are not used (thanks to bsazonov@ for detecting that!), and their tests are problematic, as they cause the notifications to be fired without the accounts actually being available in PO2TS; this fact violates an invariant that we would like to put in place wherein PO2TS only fires refresh token available notifications for accounts for which it has a refresh token. This CL simply eliminates these unused methods and their tests. TBR=msarda@chromium.org Bug: 883722 Change-Id: Ib44be7db2fbe5341588a9e7ff786f5d166d1ae6c Reviewed-on: https://chromium-review.googlesource.com/1240275 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by: Mihai Sardarescu <msarda@chromium.org> Reviewed-by: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#593546}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .