New issue
Advanced search Search tips
Starred by 2 users

Issue metadata

Status: Fixed
Closed: Jan 12
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

issue 883330

Sign in to add a comment

Issue 920208: Port to use IdentityTestEnvironment rather than SigninManager

Reported by, Jan 9 Project Member

Issue description

Should use IdentityTestEnvironment::MakePrimaryAccountAvailable() rather than SigninManager::SetAuthenticatedAccountInfo(). Please reach out if there are any complexities in the conversion.

Comment 1 by, Jan 9

Status: Started (was: Available)

Comment 2 by, Jan 12

Project Member
The following revision refers to this bug:

commit ed4f719da5ca83c0bb44d771ac618df348ad02a4
Author: Antonio Gomes <>
Date: Sat Jan 12 04:34:40 2019

[s13n] Port 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-by: Colin Blundell <>
Reviewed-by: Toni Baržić <>
Commit-Queue: Antonio Gomes <>
Cr-Commit-Position: refs/heads/master@{#622290}

Comment 3 by, Jan 12

Status: Fixed (was: Started)

Sign in to add a comment