This one requires a mapping for SigninManager::IsAllowedUsername(), blocked on 889899

> This one requires a mapping for SigninManager::IsAllowedUsername(), blocked on 889899

Oops! Too fast: That would be to move away from SigninManager, not from PO2TS. @blundell can you clarify whether leaving the dependency on SigninManager would be ok here?

Sure, let's do PO2TS for now :).

Thanks for clarifying. Assigning now...

CL here: https://chromium-review.googlesource.com/c/chromium/src/+/1338082

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

commit 945828b60437c73f8f9160831331e2adaef6b2ef
Author: Mario Sanchez Prada <mario@igalia.com>
Date: Thu Nov 15 17:52:05 2018

Convert signin_ui_util away from ProfileOAuth2TokenService

Replace usage of PO2TS APIs with calls to IdentityManager's methods.

Bug:  904412 
Change-Id: I0543f8e4c78a5b19915933082f9f9074dcb4d7d8
Reviewed-on: https://chromium-review.googlesource.com/c/1338082
Reviewed-by: David Roger <droger@chromium.org>
Commit-Queue: Mario Sanchez Prada <mario@igalia.com>
Cr-Commit-Position: refs/heads/master@{#608424}
[modify] https://crrev.com/945828b60437c73f8f9160831331e2adaef6b2ef/chrome/browser/signin/signin_ui_util.cc