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 BugsAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: aboya, cdumez, commit-queue, jlewis3, realdawei, tsavell, webkit-bug-importer, youennf
Priority: P2 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

Description Ryan Haddad 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
+
Comment 1 Ryan Haddad 2018-09-18 17:51:35 PDT 
Marked test as flaky in https://trac.webkit.org/changeset/236160/webkit
Comment 2 Radar WebKit Bug Importer 2018-09-18 17:52:29 PDT 
<rdar://problem/44584993>
Comment 3 Alexey Proskuryakov 2018-09-19 16:21:07 PDT 
Is this same as bug 181901?
Comment 4 youenn fablet 2018-12-04 14:05:28 PST 
Created attachment 356530 [details]
Change expectation
Comment 5 youenn fablet 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.
Comment 6 WebKit Commit Bot 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>
Comment 7 WebKit Commit Bot 2018-12-04 14:33:33 PST 
All reviewed patches have been landed.  Closing bug.
Comment 8 youenn fablet 2018-12-04 16:27:05 PST 
Need further investigation
Comment 9 youenn fablet 2018-12-04 20:34:12 PST 
Created attachment 356575 [details]
Patch
Comment 10 youenn fablet 2018-12-04 20:36:37 PST 
Created attachment 356576 [details]
Patch
Comment 11 youenn fablet 2018-12-04 20:37:05 PST 
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/14369
Comment 12 WebKit Commit Bot 2018-12-05 13:15:45 PST 
Comment on attachment 356576 [details]
Patch

Clearing flags on attachment: 356576

Committed r238907: <https://trac.webkit.org/changeset/238907>
Comment 13 WebKit Commit Bot 2018-12-05 13:15:47 PST 
All reviewed patches have been landed.  Closing bug.
Comment 14 Alicia Boya García 2018-12-06 03:04:30 PST 
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/14391