New issue
Advanced search Search tips

Issue 890812 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 21
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Task

Blocked on:
issue 889863

Blocking:
issue 883330



Sign in to add a comment

Convert chrome/browser/ui/webui/signin/dice_turn_sync_on_helper_unittest.cc to IdentityManager

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

Issue description

API used:
- prefs::kSigninAllowed
- SigninManager::IsAuthenticated()

 
Labels: Proj-Servicification-VendorBug
Owner: ma...@igalia.com
Status: Started (was: Available)
Taking this one
Owner: toniki...@chromium.org
Actually, I found out that Antonio is tackling this one as part of the work for crbug.com/890811, see https://crrev.com/c/1387393 => Re-assigning
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 21

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

commit a59054d082649116a5656724026a872c3ac1c4c6
Author: Antonio Gomes <tonikitoo@igalia.com>
Date: Fri Dec 21 20:59:14 2018

[s13n] Convert c/b/ui/webui/signin/dice_turn_sync_on_helper.cc to IdentityManager

SigninManager, PO2TS and AccountTrackerService are going to be an implementation
detail of the IdentityManager et al, and eventually will not be exposed to clients
out of //services/identity.
This CL converts DiceTurnSyncOnHelper and its subclasses accordingly.

The remaing work is the completely removal of the use of PO2TS in production
code, which is blocked on  bug 907901  (PO2TS::RevokeCredentials).

BUG=890811, 890812 

Change-Id: I5673c254e2ce494293627616f596945945d6ded1
Reviewed-on: https://chromium-review.googlesource.com/c/1387393
Reviewed-by: David Roger <droger@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#618594}
[modify] https://crrev.com/a59054d082649116a5656724026a872c3ac1c4c6/chrome/browser/ui/webui/signin/dice_turn_sync_on_helper.cc
[modify] https://crrev.com/a59054d082649116a5656724026a872c3ac1c4c6/chrome/browser/ui/webui/signin/dice_turn_sync_on_helper.h
[modify] https://crrev.com/a59054d082649116a5656724026a872c3ac1c4c6/chrome/browser/ui/webui/signin/dice_turn_sync_on_helper_unittest.cc

Status: Fixed (was: Started)
Missing bits are covered by bug 890811.

Sign in to add a comment