Port enterprise_platform_keys_private_api_unittest.cc to use IdentityTestEnvironment rather than SigninManager |
||
Issue descriptionShould use IdentityTestEnvironment::MakePrimaryAccountAvailable() rather than SigninManager::SetAuthenticatedAccountInfo(). Please reach out if there are any complexities in the conversion.
,
Jan 12
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ed4f719da5ca83c0bb44d771ac618df348ad02a4 commit ed4f719da5ca83c0bb44d771ac618df348ad02a4 Author: Antonio Gomes <tonikitoo@igalia.com> Date: Sat Jan 12 04:34:40 2019 [s13n] Port enterprise_platform_keys_private_api_unittest.cc way from SigninManager ... in favor of identity testing utils. SigninManager is going to be an implementation detail of IdentityManager. This CL is a straightforward move between the two APIs. BUG= 920208 Change-Id: I4372b7c81fdd229ea6a7b34b67b2c32d03e00c92 Reviewed-on: https://chromium-review.googlesource.com/c/1405728 Reviewed-by: Colin Blundell <blundell@chromium.org> Reviewed-by: Toni Baržić <tbarzic@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#622290} [modify] https://crrev.com/ed4f719da5ca83c0bb44d771ac618df348ad02a4/chrome/browser/extensions/api/enterprise_platform_keys_private/enterprise_platform_keys_private_api_unittest.cc
,
Jan 12
|
||
►
Sign in to add a comment |
||
Comment 1 by toniki...@chromium.org
, Jan 9Status: Started (was: Available)