Convert browsing_data_counter_utils.cc to use IdentityManager rather than ProfileOAuth2TokenService |
|||||
Issue descriptionCurrently blocked on missing API (see blocking bug).
,
Sep 12
,
Oct 1
Taking this one. CL here: https://chromium-review.googlesource.com/c/chromium/src/+/1255624
,
Oct 3
This now depends on crbug.com/890796 to be completely migrated (see TODO note in the CL). Setting dependency.
,
Oct 8
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7b528f414ea5f923e859e21fca15f18eab2ec98a commit 7b528f414ea5f923e859e21fca15f18eab2ec98a Author: Mario Sanchez Prada <mario@igalia.com> Date: Mon Oct 08 10:21:19 2018 Convert browsing_data_counter_utils.cc to use IdentityManager Migrate from using ProfileOAuth2TokenService and SigninManager to using the IdentityManager instead, and remove the related includes. Bug: 882862 Change-Id: I5fb46927ffba508e6987f615ed22bc1a90155607 Reviewed-on: https://chromium-review.googlesource.com/c/1255624 Reviewed-by: Joshua Bell <jsbell@chromium.org> Reviewed-by: Colin Blundell <blundell@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#597519} [modify] https://crrev.com/7b528f414ea5f923e859e21fca15f18eab2ec98a/chrome/browser/browsing_data/counters/browsing_data_counter_utils.cc
,
Oct 8
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by blundell@chromium.org
, Sep 11