Bug 189727

Summary:

[iOS] Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/fetch-cors-xhr.https.html is a flaky failure

Product:

WebKit

Reporter:

Ryan Haddad <ryanhaddad>

Component:

New Bugs

Assignee:

youenn fablet <youennf>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aboya, cdumez, commit-queue, jlewis3, realdawei, tsavell, webkit-bug-importer, youennf

Priority:

Keywords:

InRadar

Version:

Other

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=181901
https://github.com/web-platform-tests/wpt/pull/14369
https://github.com/web-platform-tests/wpt/pull/14391

Attachments:

Description	Flags
Change expectation	none
Patch	none
Patch	none

Ryan Haddad

Reported 2018-09-18 17:47:57 PDT

[iOS] Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/fetch-cors-xhr.https.html is a flaky failure This test is flaky when run on iOS 12. The diffs show that the test is hitting a harness timeout: --- /build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-cors-xhr.https-expected.txt +++ /build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-cors-xhr.https-actual.txt @@ -1,3 +1,6 @@ + -PASS Verify CORS XHR of fetch() in a Service Worker +Harness Error (TIMEOUT), message = null +TIMEOUT Verify CORS XHR of fetch() in a Service Worker Test timed out +

Attachments
Change expectation (1.81 KB, patch) 2018-12-04 14:05 PST, youenn fablet	no flags	Details Formatted Diff Diff
Patch (23.54 KB, patch) 2018-12-04 20:34 PST, youenn fablet	no flags	Details Formatted Diff Diff
Patch (23.27 KB, patch) 2018-12-04 20:36 PST, youenn fablet	no flags	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Ryan Haddad

Comment 1 2018-09-18 17:51:35 PDT

Marked test as flaky in https://trac.webkit.org/changeset/236160/webkit

Radar WebKit Bug Importer

Comment 2 2018-09-18 17:52:29 PDT

<rdar://problem/44584993>

Alexey Proskuryakov

Comment 3 2018-09-19 16:21:07 PDT

Is this same as bug 181901?

youenn fablet

Comment 4 2018-12-04 14:05:28 PST

Created attachment 356530 [details] Change expectation

youenn fablet

Comment 5 2018-12-04 14:07:23 PST

(In reply to youenn fablet from comment #4) > Created attachment 356530 [details] > Change expectation This will allow making the test showing as less flaky than it is. The timeout issue still remains.

WebKit Commit Bot

Comment 6 2018-12-04 14:33:31 PST

Comment on attachment 356530 [details] Change expectation Clearing flags on attachment: 356530 Committed r238875: <https://trac.webkit.org/changeset/238875>

WebKit Commit Bot

Comment 7 2018-12-04 14:33:33 PST

All reviewed patches have been landed. Closing bug.

youenn fablet

Comment 8 2018-12-04 16:27:05 PST

Need further investigation

youenn fablet

Comment 9 2018-12-04 20:34:12 PST

Created attachment 356575 [details] Patch