Bug 150363 – http/tests/xmlhttprequest/timeout/xmlhttprequest-timeout-twice.html is flaky

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Alexey Proskuryakov]

http/tests/xmlhttprequest/timeout/xmlhttprequest-timeout-twice.html is flaky:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fxmlhttprequest%2Ftimeout%2Fxmlhttprequest-timeout-twice.html

 PASS XHR2 Timeout Property Tests 
-PASS XHR2 Timeout Property Tests 1 
-PASS XHR2 Timeout Property Tests 2 
+FAIL XHR2 Timeout Property Tests 1 assert_equals: load fires normally with same timeout set twice, original timeout at 1000, reset at 400 to 1000 expected "load" but got "timeout"(stack: assert@http://127.0.0.1:8000/w3c/resources/testharness.js:2060:37
+assert_equals@http://127.0.0.1:8000/w3c/resources/testharness.js:699:15
+http://127.0.0.1:8000/xmlhttprequest/timeout/xmlhttprequest-timeout-runner.js:7:40
+step@http://127.0.0.1:8000/w3c/resources/testharness.js:1160:30
+test@http://127.0.0.1:8000/w3c/resources/testharness.js:443:22
+testResultCallbackHandler@http://127.0.0.1:8000/xmlhttprequest/timeout/xmlhttprequest-timeout-runner.js:7:13)

[youenn fablet]

If I am not mistaken, WPT testsuite contains the same tests.
It might be better to use WPT latest version and import them in LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest.

I'll try to do that once bug 149656 is done.

Also, IIRC, timeout values in the WPT tests have been increased.

[Ryan Haddad]

Created attachment 263611 [details]
Patch

This is failing frequently today on Yosemite. Marking as flaky.

[WebKit Commit Bot]

Comment on attachment 263611 [details]
Patch

Clearing flags on attachment: 263611

Committed r191375: <http://trac.webkit.org/changeset/191375>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Alexey Proskuryakov]

Re-opening for investigation.

[youenn fablet]

Fixed as part of bug 151091