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

Issue 818324 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug



Sign in to add a comment

"external/wpt/content-security-policy/embedded-enforcement/required_csp-header.html" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Mar 2 2018

Issue description

"external/wpt/content-security-policy/embedded-enforcement/required_csp-header.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=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXQsSBUZsYWtlIlJleHRlcm5hbC93cHQvY29udGVudC1zZWN1cml0eS1wb2xpY3kvZW1iZWRkZWQtZW5mb3JjZW1lbnQvcmVxdWlyZWRfY3NwLWhlYWRlci5odG1sDA.

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
 
Components: Blink>SecurityFeature>ContentSecurityPolicy
Labels: -Sheriff-Chromium
Labels: OS-Linux OS-Windows
Owner: andypaicu@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 4 by chromium...@appspot.gserviceaccount.com, Apr 10 2018

Labels: Sheriff-Chromium
Detected 3 new flakes for test/step "external/wpt/content-security-policy/embedded-enforcement/required_csp-header.html". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXQsSBUZsYWtlIlJleHRlcm5hbC93cHQvY29udGVudC1zZWN1cml0eS1wb2xpY3kvZW1iZWRkZWQtZW5mb3JjZW1lbnQvcmVxdWlyZWRfY3NwLWhlYWRlci5odG1sDA. This message was posted automatically by the chromium-try-flakes app. Since flakiness is ongoing, the issue was moved back into Sheriff Bug Queue (unless already there).
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 11 2018

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

commit e81b180a7ad87e07c103d2dce227cc666de9d148
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Wed Apr 11 22:57:11 2018

Build sheriff: disable a number of flaky layout tests on Linux.

TBR=junov@chromium.org

Bug:  831701 
Bug:  831686 
Bug:  831685 
Bug:  831673 
Bug:  831496 
Bug:  831482 
Bug:  831249 
Bug:  831230 
Bug: 829952
Bug:  829938 
Bug: 818426
Bug: 818324
Bug:  810437 
Change-Id: Id3f657e2c5d2d46456892069e61730689a5f733b
Reviewed-on: https://chromium-review.googlesource.com/1008411
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549964}
[modify] https://crrev.com/e81b180a7ad87e07c103d2dce227cc666de9d148/third_party/WebKit/LayoutTests/TestExpectations

Labels: -Sheriff-Chromium
Project Member

Comment 7 by bugdroid1@chromium.org, Apr 12 2018

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

commit bc286e3b63877aac5f87fce42d11f4b728d4b53f
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Thu Apr 12 20:35:58 2018

Ignore timeout results on some flaky layout tests.

* external/wpt/html/syntax/parsing/html5lib_template.html?run_type=uri
* virtual/gpu-rasterization/images/cross-fade-overflow-position.html

TBR=shend@chromium.org,schenney@chromium.org

Bug: 818324
Bug: 831993
Change-Id: I011bd7deb75b48c7572643a29d43f62a8a69ea6e
Reviewed-on: https://chromium-review.googlesource.com/1010908
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550340}
[modify] https://crrev.com/bc286e3b63877aac5f87fce42d11f4b728d4b53f/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 8 by bugdroid1@chromium.org, Apr 17 2018

Labels: merge-merged-testbranch
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bc286e3b63877aac5f87fce42d11f4b728d4b53f

commit bc286e3b63877aac5f87fce42d11f4b728d4b53f
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Thu Apr 12 20:35:58 2018

Ignore timeout results on some flaky layout tests.

* external/wpt/html/syntax/parsing/html5lib_template.html?run_type=uri
* virtual/gpu-rasterization/images/cross-fade-overflow-position.html

TBR=shend@chromium.org,schenney@chromium.org

Bug: 818324
Bug: 831993
Change-Id: I011bd7deb75b48c7572643a29d43f62a8a69ea6e
Reviewed-on: https://chromium-review.googlesource.com/1010908
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550340}
[modify] https://crrev.com/bc286e3b63877aac5f87fce42d11f4b728d4b53f/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 9 by bugdroid1@chromium.org, May 4 2018

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

commit 732cc14175c56476b0b172f6d398518fee7912a0
Author: Robert Ma <robertma@chromium.org>
Date: Fri May 04 15:26:17 2018

Remove some timeout expectations after a bug fix

These tests already declare to be slow (long timeout), but due to a bug
we previously didn't parse the manifest correctly so we missed them.
https://crrev.com/c/1043160 fixed the issue so we can now remove these
expectations.

TBR=tkent

Bug:  803943 , 818324, 803558,  831975 
Change-Id: I8f978d025e026b77dd6f42ade6de8e283886c024
Reviewed-on: https://chromium-review.googlesource.com/1044293
Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#556062}
[modify] https://crrev.com/732cc14175c56476b0b172f6d398518fee7912a0/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 10 by chromium...@appspot.gserviceaccount.com, May 29 2018

Labels: Sheriff-Chromium
Detected 3 new flakes for test/step "external/wpt/content-security-policy/embedded-enforcement/required_csp-header.html". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXQsSBUZsYWtlIlJleHRlcm5hbC93cHQvY29udGVudC1zZWN1cml0eS1wb2xpY3kvZW1iZWRkZWQtZW5mb3JjZW1lbnQvcmVxdWlyZWRfY3NwLWhlYWRlci5odG1sDA. This message was posted automatically by the chromium-try-flakes app. Since flakiness is ongoing, the issue was moved back into Sheriff Bug Queue (unless already there).
Project Member

Comment 11 by chromium...@appspot.gserviceaccount.com, Jun 2 2018

Detected 4 new flakes for test/step "external/wpt/content-security-policy/embedded-enforcement/required_csp-header.html". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXQsSBUZsYWtlIlJleHRlcm5hbC93cHQvY29udGVudC1zZWN1cml0eS1wb2xpY3kvZW1iZWRkZWQtZW5mb3JjZW1lbnQvcmVxdWlyZWRfY3NwLWhlYWRlci5odG1sDA. This message was posted automatically by the chromium-try-flakes app.
Labels: -Sheriff-Chromium
Send a CL to set test expectations (Timeouts) on Mac as well.
Project Member

Comment 13 by bugdroid1@chromium.org, Jun 4 2018

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

commit 29f47b22a63833e55fd4bb76c915dbdf458cc527
Author: Dominic Battre <battre@chromium.org>
Date: Mon Jun 04 13:24:36 2018

required_csp-header.html times out on Mac as well

TBR=andypaicu@chromium.org

Bug: 818324
Change-Id: I4a82ff5fe5a595a24b02500d246597a0fe9a9ef7
Reviewed-on: https://chromium-review.googlesource.com/1084933
Reviewed-by: Dominic Battré <battre@chromium.org>
Reviewed-by: Andy Paicu <andypaicu@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564075}
[modify] https://crrev.com/29f47b22a63833e55fd4bb76c915dbdf458cc527/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 14 by bugdroid1@chromium.org, Jul 6

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

commit fa8b12920e8c2042e47816d004b113c155e81aee
Author: Alexis Hetu <sugoi@google.com>
Date: Fri Jul 06 14:14:32 2018

Attempt to unmark some tests as slow/timeout

Now that SwiftShader is on Linux/Windows/MacOS and that a recent
performance improvement has been landed in SwiftShader, verify
which tests still require the Slow/Timeout markers and which don't.

TBR=kbr@chromium.org

Bug:chromium:24182  chromium:433711  chromium:763197  chromium:311482   chromium:243871   chromium:664857   chromium:9798   chromium:237270   chromium:241576   chromium:241869   chromium:246749   chromium:535478   chromium:363029   chromium:364225   chromium:552556   chromium:570656   chromium:584807  chromium:614910  chromium:791659   chromium:726075   chromium:808153  chromium:816045  chromium:693568  chromium:626703 chromium:703533 chromium:786641  chromium:799137   chromium:831686   chromium:831230  chromium:818324  chromium:810437   chromium:847205  chromium:848799  chromium:828962   chromium:849284   chromium:855055 

Change-Id: I5d36d20bd87b234fefe4da3ea7e4af039c0188cb
Reviewed-on: https://chromium-review.googlesource.com/1102341
Reviewed-by: Alexis Hétu <sugoi@chromium.org>
Commit-Queue: Alexis Hétu <sugoi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572962}
[modify] https://crrev.com/fa8b12920e8c2042e47816d004b113c155e81aee/third_party/WebKit/LayoutTests/SlowTests
[modify] https://crrev.com/fa8b12920e8c2042e47816d004b113c155e81aee/third_party/WebKit/LayoutTests/TestExpectations

Labels: Sheriff-Chromium
Detected 4 new flakes for test/step "external/wpt/content-security-policy/embedded-enforcement/required_csp-header.html". To see the actual flakes, please visit https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyXQsSBUZsYWtlIlJleHRlcm5hbC93cHQvY29udGVudC1zZWN1cml0eS1wb2xpY3kvZW1iZWRkZWQtZW5mb3JjZW1lbnQvcmVxdWlyZWRfY3NwLWhlYWRlci5odG1sDA. This message was posted automatically by the chromium-try-flakes app. Since flakiness is ongoing, the issue was moved back into Sheriff Bug Queue (unless already there).
Labels: -Sheriff-Chromium
Disabling again: https://chromium-review.googlesource.com/c/chromium/src/+/1134052
Project Member

Comment 17 by bugdroid1@chromium.org, Jul 11

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

commit a18e8a4a19f2f5dd2d52b8d715da8fa61a3b9654
Author: Joshua Pawlicki <waffles@chromium.org>
Date: Wed Jul 11 20:47:42 2018

Re-mark csp test as flaky (was speculatively unmarked).

Bug: 818324
Change-Id: Ie3b8afb30a9a84c124cbe22978e1dd491dbe84e4
TBR: dpranke@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1134052
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574322}
[modify] https://crrev.com/a18e8a4a19f2f5dd2d52b8d715da8fa61a3b9654/third_party/WebKit/LayoutTests/TestExpectations

Sign in to add a comment