New issue
Advanced search Search tips

Issue 922742 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Yesterday
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: ----

Blocking:
issue 912170



Sign in to add a comment

Replace use of AccountTrackerService in chrome/browser/android/signin/signin_manager_android.cc

Project Member Reported by dxie@google.com, Jan 16 (6 days ago)

Issue description


 

Comment 1 by dxie@google.com, Jan 16 (6 days ago)

Components: Internals>Services>Identity
Labels: Proj-Servicification OS-Windows OS-Linux OS-Mac Proj-Servicification-VendorBug OS-Android Pri-2
Status: Available (was: Untriaged)

Comment 2 by dxie@google.com, Jan 16 (6 days ago)

Blocking: 912170
Labels: -OS-Linux -OS-Windows -OS-Mac
"API replacement summary:
- AccountTrackerService::FindAccountInfoByEmail => IdentityManager::FindAccountInfoForAccountWithRefreshTokenByEmailAddress
"

Comment 3 by abhij...@igalia.com, Jan 19 (3 days ago)

Owner: abhij...@igalia.com

Comment 4 by abhij...@igalia.com, Yesterday (46 hours ago)

Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Yesterday (25 hours ago)

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

commit c382936c222d0f1dc732fbf478c3e47bffc68c98
Author: Abhijeet Kandalkar <abhijeet@igalia.com>
Date: Tue Jan 22 04:29:42 2019

Replace use of AccountTrackerService form signin_manager_android.cc

This CL replaces AccountTrackerService::FindAccountInfoByEmail with
IdentityManager::FindAccountInfoForAccountWithRefreshTokenByEmailAddress.

Remove unused headers.

Bug:  922742 
Change-Id: Ided55ebf6761543d158601bb52b0095b7b12a55f
Reviewed-on: https://chromium-review.googlesource.com/c/1424104
Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com>
Reviewed-by: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624725}
[modify] https://crrev.com/c382936c222d0f1dc732fbf478c3e47bffc68c98/chrome/browser/android/signin/signin_manager_android.cc

Comment 6 by abhij...@igalia.com, Yesterday (25 hours ago)

Status: Fixed (was: Started)

Sign in to add a comment