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

Issue 663872 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Leaves the project on 2018/03/02
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 601332



Sign in to add a comment

4 tests in http/tests/fetch/serviceworker-proxied/thorough/ are flaky when run in random order

Project Member Reported by jeffcarp@chromium.org, Nov 9 2016

Issue description

The tests:

http/tests/fetch/serviceworker-proxied/thorough/cookie-nocors.html
http/tests/fetch/serviceworker-proxied/thorough/cors-preflight-other-https.html
http/tests/fetch/serviceworker-proxied/thorough/redirect-base-https-other-https.html
http/tests/fetch/serviceworker-proxied/thorough/redirect-credentials-base-https-other-https.html

appear to be flaky when run in random order. Steps to reproduce:

This should pass consistently:
Tools/Scripts/run-webkit-tests http/tests/fetch/serviceworker-proxied/thorough/

This should fail intermittently:
Tools/Scripts/run-webkit-tests http/tests/fetch/serviceworker-proxied/thorough/ --order=random

We're working on removing order dependency from Blink layout tests. More context here:
https://docs.google.com/document/d/11-628Ny6hY__kfIv__zVD0m3rA0OB9tQLLC_gJI_PNg/edit

 

Comment 1 by falken@chromium.org, Nov 11 2016

Components: Blink>Network>FetchAPI
Owner: tyoshino@chromium.org
Status: Assigned (was: Untriaged)
tyoshino: Could network API team take this? Feel free to kick back to Untriaged otherwise.
Description: Show this description
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 7 2017

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

commit b034403ab50b3a27bda35d6cf72efec57f2f1217
Author: qyearsley <qyearsley@chromium.org>
Date: Tue Feb 07 17:50:47 2017

Remove flaky expectations with update-test-expectations script

The update-test-expectations script is meant to remove
flaky expectations when it appears that according to the
most recent results on the flakiness dashboard for that test,
the test is no longer flaky.

BUG= 491764 , 498539 , 575766 , 600248 ,658305, 663838 , 663840 ,663848, 663851 , 663853 , 663872 , 663874 , 663877 , 663879 ,664817,664839, 664840 , 664841 , 664842 , 664846 , 664850 , 664855 , 664856 , 666991 , 671480 , 671618 , 672204 , 673296 , 673632 , 674720 ,674858, 678488 , 683800 

Review-Url: https://codereview.chromium.org/2679543005
Cr-Commit-Position: refs/heads/master@{#448663}

[modify] https://crrev.com/b034403ab50b3a27bda35d6cf72efec57f2f1217/third_party/WebKit/LayoutTests/TestExpectations

Comment 4 by falken@chromium.org, Apr 14 2017

Status: WontFix (was: Assigned)
Seems obsolete from comment 3.

Sign in to add a comment