Bug 178172 – Re-sync XMLHttpRequests Web-Platform-Tests

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

[Chris Dumez]

Re-sync XMLHttpRequests Web-Platform-Tests.

[Chris Dumez]

Created attachment 323426 [details]
Patch

[Chris Dumez]

Created attachment 323427 [details]
Patch

[youenn fablet]

Comment on attachment 323427 [details]
Patch

Some tests should be fixed to be actually useful.
Use common/get-host-info.sub.js for that purpose.

View in context: https://bugs.webkit.org/attachment.cgi?id=323427&action=review

> LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest/open-url-worker-origin.htm:20
> +        var worker = new Worker("resources/workerxhr-origin-referrer.js")

Can we fix /workerxhr-origin-referrer.js to not use www2?

> LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest/send-after-setting-document-domain.htm:12
> +      var test_base_url = location.protocol+'//www2.'+location.host+"/XMLHttpRequest/resources/",

Test should be fixed upstream

> LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest/send-authentication-basic-cors-not-enabled.htm:16
> +          urlstart = 'www1.'+location.host + location.pathname.replace(/\/[^\/]*$/, '/')

Test should be fixed upstreamed.

> LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest/send-authentication-basic-cors.htm:16
> +          urlstart = 'www1.'+location.host + location.pathname.replace(/\/[^\/]*$/, '/')

Ditto.

> LayoutTests/imported/w3c/web-platform-tests/XMLHttpRequest/send-authentication-basic-expected.txt:3
> +Note: this test will only work as expected once per browsing session. Restart browser to re-test.

Can it make things flaky?

[Build Bot]

Comment on attachment 323427 [details]
Patch

Attachment 323427 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4825914

New failing tests:
imported/w3c/web-platform-tests/XMLHttpRequest/preserve-ua-header-on-redirect.htm

[Build Bot]

Created attachment 323434 [details]
Archive of layout-test-results from ews107 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 323427 [details]
Patch

Attachment 323427 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4826054

New failing tests:
imported/w3c/web-platform-tests/XMLHttpRequest/preserve-ua-header-on-redirect.htm

[Build Bot]

Created attachment 323436 [details]
Archive of layout-test-results from ews103 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 323427 [details]
Patch

Attachment 323427 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4826153

New failing tests:
imported/w3c/web-platform-tests/XMLHttpRequest/preserve-ua-header-on-redirect.htm

[Build Bot]

Created attachment 323441 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.6

[Chris Dumez]

Created attachment 323442 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 323442 [details]
Patch

Clearing flags on attachment: 323442

Committed r223203: <https://trac.webkit.org/changeset/223203>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/34939253>