New issue
Advanced search Search tips

Issue 891906 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 5
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: ExtensionServiceTestSupervised.ExtensionApprovalBeforeInstallation



Sign in to add a comment

ExtensionServiceTestSupervised.ExtensionApprovalBeforeInstallation is flaky

Project Member Reported by Findit, Oct 3

Issue description

Components: Platform>Extensions
Owner: mastiz@chromium.org
Status: Assigned (was: Untriaged)
This test (and others) seems to first call InitServices and kick of creating services, and afterwards tries to call ScopedFeatureList::InitAndEnableFeature, which races with some of these services trying to check if a feature is enabled.

"Caused" by https://chromium-review.googlesource.com/c/chromium/src/+/1257912 but this was always a time bomb waiting to happen...
Labels: -Sheriff-Chromium
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 5

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

commit a68eea5eb428d5f7c1aad7ced339016251984e36
Author: Mikel Astiz <mastiz@chromium.org>
Date: Fri Oct 05 07:59:14 2018

Fix TSAN flakiness due to late ScopedFeatureList

Feature toggles cannot change during runtime, so let's make sure the
final values are fixed before a profile is created for tests. This fixes
flakiness in TSAN bots due to recent unrelated changes, where a
KeyedService's backend thread reads a feature toggle (although the
feature is unrelated to this test suite).

TBR=rockot@chromium.org

Bug:  891906 
Change-Id: I385c8d5ff38c660d84465cbaaa1bcbeca6b9482a
Reviewed-on: https://chromium-review.googlesource.com/c/1264161
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597026}
[modify] https://crrev.com/a68eea5eb428d5f7c1aad7ced339016251984e36/chrome/browser/extensions/extension_service_sync_unittest.cc

Status: Fixed (was: Started)
I'll assume the patch above fixes the issue, let's reopen if that's not the case.

Sign in to add a comment