New issue
Advanced search Search tips

Issue 909712 link

Starred by 2 users
Status: Fixed
Owner:
sdefresne@chromium.org
Closed: Nov 29
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

NOTIMPLEMENTED() in PrimaryAccountMutatorImpl

Project Member Reported by skobes@chromium.org, Nov 28 
Chrome Version: ToT
OS Version: gLinux
URLs (if applicable): about:blank

What steps will reproduce the problem?
1. Run chrome

What is the expected result?
No logspam

What happens instead of that?
[112499:112499:1128/092348.883214:ERROR:primary_account_mutator_impl.cc(103)] Not implemented reached in virtual bool identity::PrimaryAccountMutatorImpl::LegacyIsPrimaryAccountAuthInProgress() const

This method was added in r602467.

Comment 1 by sdefresne@chromium.org, Nov 28

Status: Started (was: Unconfirmed)
This is due to incorrect migration to new API even though it has not been implemented yet (https://chromium-review.googlesource.com/c/1346467). I've created a CL to revert the conversion.

However, this conversion should have failed if the feature had properly been tested (LegacyIsPrimaryAccountAuthInProgress always returns false, so the code should have failed in some case).

Comment 2 by sdefresne@chromium.org, Nov 29

Status: Fixed (was: Started)

Comment 3 by sdefresne@chromium.org, Nov 29 

The offending CL has been reverted.
►

Sign in to add a comment