New issue
Advanced search Search tips

Issue 906579 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 21
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 883318



Sign in to add a comment

Port identity_api.cc away from observing ProfileOAuth2TokenService to observing IdentityManager

Project Member Reported by blundell@google.com, Nov 19

Issue description

Should be totally straightforward.
 
Blocking: 883318
Owner: toniki...@chromium.org
Status: Started (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 21

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

commit d22ab991da0abcd793f0cbbeb79ae29d9cbcb831
Author: Antonio Gomes <tonikitoo@igalia.com>
Date: Wed Nov 21 13:35:07 2018

[s13n] Port identity_api.cc away from observing ProfileOAuth2TokenService

... to observing IdentityManager

CL also gets rid of the AccountTrackerService usage, driven-by.

BUG= 906579 

Change-Id: I1ec47f5817a665995a5c1e4c2a6ed21a49756f3e
Reviewed-on: https://chromium-review.googlesource.com/c/1340890
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#610032}
[modify] https://crrev.com/d22ab991da0abcd793f0cbbeb79ae29d9cbcb831/chrome/browser/extensions/api/identity/identity_api.cc
[modify] https://crrev.com/d22ab991da0abcd793f0cbbeb79ae29d9cbcb831/chrome/browser/extensions/api/identity/identity_api.h

Status: Fixed (was: Started)

Sign in to add a comment