New issue
Advanced search Search tips

Issue 920215 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 16
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 883330



Sign in to add a comment

Port cloud_policy_manager_browsertest.cc to use IdentityManager rather than SigninManager

Project Member Reported by blundell@chromium.org, Jan 9

Issue description

Specifically, it should use PrimaryAccountMutator to set the primary account information.

 
Owner: je_julie.kim@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 15

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2449cbd81698ee25bf5d0fce27b6d5c65bceb2e6

commit 2449cbd81698ee25bf5d0fce27b6d5c65bceb2e6
Author: Julie Jeongeun Kim <jkim@igalia.com>
Date: Tue Jan 15 15:29:04 2019

[s13n] Move CloudPolicyManagerTest away from using SigninManager

This CL makes CloudPolicyManagerTest use IdentityManager instead
of SigninManager.

SigninManager is going to be an implementation detail of
IdentityManager.

Bug:  920215 
Change-Id: I9f379321fa02133349113c96f60bab5aa27fe7bb
Reviewed-on: https://chromium-review.googlesource.com/c/1411195
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com>
Cr-Commit-Position: refs/heads/master@{#622865}
[modify] https://crrev.com/2449cbd81698ee25bf5d0fce27b6d5c65bceb2e6/chrome/browser/policy/cloud/cloud_policy_manager_browsertest.cc

Status: Fixed (was: Started)

Sign in to add a comment