New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 860458 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug

Blocking:
issue 801985
issue 866453



Sign in to add a comment

Browser tests crashe when ProfileSyncService instantiates InstanceIDProfileService

Project Member Reported by melandory@chromium.org, Jul 5

Issue description

Browser tests crashes when ProfileSyncService instantiates InstanceIDProfileService

Short CL to reproduce the issue https://chromium-review.googlesource.com/c/chromium/src/+/1102421

Suspect of what's happening.

Something builds a ProfileSyncService instance immediately or *very* shortly after on profile creation, which in turn builds an InstanceIDProfileService instance.

All failing test suites hook the InstanceIDProfileServiceFactory in their BrowserTestBase::SetUpOnMainThread() override. That runs just *after* the browser (w/ the profile) is created, which means that per this CL, an instance already exists by the time these tests are trying to hook the creation function.

One thing to try is to set the testing factory hook in SetUp() instead of SetUpOnMainThread().
 
Components: Services>Invalidation Services>Sync
Labels: OS-Chrome OS-Linux OS-Mac OS-Windows
My guess is that those tests set up their testing factory stuff after the real ProfileSyncService (and thus InstanceIDProfileService) has already been created.
The beautifully-named BrowserContextDependencyManager::RegisterWillCreateBrowserContextServicesCallbackForTesting should solve this.
Blocking: 801985
Description: Show this description
Description: Show this description
Blocking: 866453
Sync triage ping. Is there any updates?
whom are you pinging and for what?
myself, I've just pasted "standard" text, not realizing that it'll look strange :)
We've discuss such issues on the triage meeting with the outcome that pinging doesn't do harm.
Status: Fixed (was: Available)

Sign in to add a comment