New issue
Advanced search Search tips
Starred by 3 users

Issue metadata

Status: Fixed
Closed: Dec 14
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Sign in to add a comment

Issue 903718: Develop IdentityManager::Observer equivalent to GaiaCookieManagerService::Observer::OnAddAccountToCookieCompleted()

Reported by, Nov 9 Project Member

Issue description

Should come with unittests similar to the ones for the other APIs in IdentityManager::Observer.

Comment 1 by, Nov 9

Blocking: 893133

Comment 2 by, Nov 9

Labels: -Pri-2 Pri-1
Bumping this up to P1 as it's blocking the conversion of the SigninTracker unittest away from direct usage of PO2TS/SigninManager and is thus blocking the completion of the current milestone of removing all direct usage of those classes. Camille, would you be interested in taking this on as part of the SigninTracker work?

We can call the API OnGaiaAccountAddedToCookie() for consistency with the sibling observer API OnGaiaAccountsInCookieUpdated().

Comment 3 by, Nov 29

Blocking: 909715
Status: Assigned (was: Available)
Antonio is going to take this on as it's blocking a conversion that he's looking to do. CC'ing Lowell, who will be managing the GaiaCookieManagerService conversion over all Real Soon Now (TM). Antonio, when you send out the CL, could you send it to both me and Lowell for review? Thanks!

Comment 4 by, Dec 11

Blocking: 859882

Comment 5 by, Dec 11

Blocking: 797931

Comment 6 by, Dec 13

Status: Started (was: Assigned)

Comment 7 by, Dec 13

Labels: Proj-Servicification-VendorBug

Comment 8 by, Dec 14

Project Member
The following revision refers to this bug:

commit 053d9abfe3ccad58e14052dd5fc2b5a5dd7906cd
Author: Antonio Gomes <>
Date: Fri Dec 14 17:01:01 2018

Develop IdentityManager::Observer equivalent to GCMS::Observer::OnAddAccountToCookieCompleted

CL adds an equivalent hook to
in IdentityManager::Observer.

This is part of the effort to allow GCMS::Observer be replaced by
IdentityManager::Observer. No functionality changes expected.

The unittests aim at ensuring the hook is called through
Observers of IdentityManager in case of successful or failing session

BUG= 903718 

Change-Id: Id08c4515171f3c21e134c2c129593e8977bd7e11
Commit-Queue: Antonio Gomes <>
Reviewed-by: Colin Blundell <>
Cr-Commit-Position: refs/heads/master@{#616713}

Comment 9 by, Dec 14

Status: Fixed (was: Started)

Comment 10 by, Jan 31

 Issue 926871  has been merged into this issue.

Sign in to add a comment