New issue
Advanced search Search tips

Issue 922764 link

Starred by 2 users

Issue metadata

Status: Available
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Mac
Pri: 2
Type: ----

Blocked on:
issue 922474

Blocking:
issue 921063
issue 922476



Sign in to add a comment

Replace use of AccountTrackerService in chrome/browser/safe_browsing/advanced_protection_status_manager_unittest.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)

Blockedon: 922474
Blocking: 921063 922476
"API replacement summary:
- AccountTrackerService::SetIsAdvancedProtectionAccount => AccountsMutator::SetIsAdvancedProtectionAccount
- AccountTrackerService::RemoveAccount => AccountsMutator::RemoveAccount
"

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

Owner: abhij...@igalia.com

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

Hi Daniel,
Regarding #c2, There is no method name SetIsAdvancedProtectionAccount in AccountsMutator. If I understood correctly, you want to refer AccountsMutator::UpdateAccountInfo() here.

Comment 5 Deleted

Comment 6 by toniki...@chromium.org, Yesterday (36 hours ago)

ok, I mis-read the question (hence the deleted comment).

I believe abhijeet's question is pertinent.

Sign in to add a comment