New issue
Advanced search Search tips

Issue 884445 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"virtual/*/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Sep 15

Issue description

"virtual/video-surface-layer/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyiAELEgVGbGFrZSJ9dmlydHVhbC92aWRlby1zdXJmYWNlLWxheWVyL2V4dGVybmFsL3dwdC9mZWF0dXJlLXBvbGljeS9mZWF0dXJlLXBvbGljeS1uZXN0ZWQtaGVhZGVyLXBvbGljeS1kaXNhbGxvd2VkLWZvci1hbGwuaHR0cHMuc3ViLmh0bWwM.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Owner: iclell...@chromium.org
Status: Assigned (was: Untriaged)
Summary: "virtual/*/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html" is flaky (was: "virtual/video-surface-layer/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html" is flaky)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 18

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

commit c49c9875450e79a5be30e9f9a33ba83c0ff6fbc4
Author: Asanka Herath <asanka@chromium.org>
Date: Tue Sep 18 17:06:10 2018

Update WPT test expectations for some feature-policy tests

Both
virtual/video-surface-layer/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html
and
virtual/unified-autoplay/external/wpt/feature-policy/feature-policy-nested-header-policy-disallowed-for-all.https.sub.html
are timing out in a flaky manner.

They were added in https://chromium-review.googlesource.com/887324
around February of 2018. Started flaking out (as far as the dashboard
knows) June 6, 2018. The most recent and more frequent flakes started
around September 12, 2018. No clear culprit found.

Bug: 884445
Change-Id: Ib26f9c51eaa41224aa964156e5b26c0174739008
Reviewed-on: https://chromium-review.googlesource.com/1231074
Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Asanka Herath <asanka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#592085}
[modify] https://crrev.com/c49c9875450e79a5be30e9f9a33ba83c0ff6fbc4/third_party/WebKit/LayoutTests/TestExpectations

Labels: -Sheriff-Chromium
Removing from sheriff queue.
Status: Started (was: Assigned)
Looks like the test is occasionally taking >6s to complete (and therefore timing out,) because it ends up loading 42 subframes, 7-at-a-time in promise tests, which necessarily run sequentially.

I'll look at doing all of the tests in parallel, or else splitting this into 2+ smaller tests.

Sign in to add a comment