New issue
Advanced search Search tips

Issue 890772 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 18
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task

Blocking:
issue 883330



Sign in to add a comment

Convert chrome/browser/chromeos/arc/arc_support_host.cc to IdentityManager

Project Member Reported by sdefresne@chromium.org, Oct 1

Issue description

API used:
- SigninManager::IsAuthenticated()

 
Owner: svil...@igalia.com
Status: Started (was: Available)
Checking this...
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 18

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

commit 55e3f84e68f754ac02daa9fb2863f28cdc400afa
Author: Sergio Villar Senin <svillar@igalia.com>
Date: Thu Oct 18 09:40:39 2018

Convert chrome/browser/chromeos/arc/arc_support_host.cc to IdentityManager

Currently using the SigninManager API to get the primary
(authenticated) account. Migrated it to IdentityManager so it could
eventually use the identity service.

Bug:  890772 
Change-Id: I22cadc9256589885c002a0cabe1e3a174b0504ad
Reviewed-on: https://chromium-review.googlesource.com/c/1282973
Reviewed-by: Colin Blundell <blundell@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Commit-Queue: Sergio Villar <svillar@igalia.com>
Cr-Commit-Position: refs/heads/master@{#600704}
[modify] https://crrev.com/55e3f84e68f754ac02daa9fb2863f28cdc400afa/chrome/browser/chromeos/arc/arc_support_host.cc
[modify] https://crrev.com/55e3f84e68f754ac02daa9fb2863f28cdc400afa/chrome/browser/chromeos/arc/arc_support_host_unittest.cc

Status: Fixed (was: Started)
Closing

Sign in to add a comment