API used - SigninManagerBase::GetAuthenticatedAccountId
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/240ba84a269c3fa70a4aed31afc2ae3abc5592b3 commit 240ba84a269c3fa70a4aed31afc2ae3abc5592b3 Author: Antonio Gomes <tonikitoo@igalia.com> Date: Fri Nov 23 12:34:42 2018 [s13n] Convert permission_request_creator_apiary.cc to IdentityManager Similarly to [1], this CL replaces the Use of OAuth2TokenService, OAuth2TokenService::Consumer and SigninManager APIs with the corresponding IdentityManager counterparts. The unittests could not be only minimally adapted, and converted on a follow up pass, given that CL needed to replace the uses of FakeProfileOAuth2TokenService with IdentityTestEnvironment upfront. [1] https://crrev.com/c/1348670 BUG= 907526 Change-Id: I59c2a49d20341ddadff15a28c7ad81e3056161e6 Reviewed-on: https://chromium-review.googlesource.com/c/1348671 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by: Marc Treib <treib@chromium.org> Reviewed-by: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#610581} [modify] https://crrev.com/240ba84a269c3fa70a4aed31afc2ae3abc5592b3/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc [modify] https://crrev.com/240ba84a269c3fa70a4aed31afc2ae3abc5592b3/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.h [modify] https://crrev.com/240ba84a269c3fa70a4aed31afc2ae3abc5592b3/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6 commit d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6 Author: Antonio Gomes <tonikitoo@igalia.com> Date: Tue Jan 22 18:49:32 2019 Use identity::PrimaryAccountAccessTokenFetcher SafeSearchURLReporter & PermissionRequestCreatorApiary Since production code of both classes operate solely fetching the access token of the primary account, this CL: - uses the class designed for the task in both classes: PrimaryAccountAccessTokenFetcher. - changes the both classes unittests to reflect the production code. BUG= 907529 , 907526 Change-Id: Ie85a98ca5e44dff6d63f33a4b3de6a16b3cc92f1 Reviewed-on: https://chromium-review.googlesource.com/c/1426759 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by: Marc Treib <treib@chromium.org> Auto-Submit: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#624846} [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.cc [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary.h [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/child_accounts/permission_request_creator_apiary_unittest.cc [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/experimental/safe_search_url_reporter.cc [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/experimental/safe_search_url_reporter.h [modify] https://crrev.com/d91eed89cc6a3d3cbe61104f1f6dcb29336b4ef6/chrome/browser/supervised_user/experimental/safe_search_url_reporter_unittest.cc
Comment 1 by toniki...@chromium.org
, Nov 21Status: Started (was: Available)