1478743 - Fix some standards compliance issues with XHR

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Description

[Boris Zbarsky [:bzbarsky]]

I was in this code for unrelated reasons and saw some things that were a bit broken...  fixes coming up.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Add some test coverage for setting xhr.responseType on workers

Most of the change is just making the responseType wpt run on workers and
annotating the resulting failures.

The change to the initial value of mResponseType is a drive-by fix for an
easy-to-fix issue the test caught.  There is a corresponding mochitest fix to
fix our incorrect test for the behavior.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Add some test coverage for calling xhr.overrideMimeType() in done state in workers

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: part 3. Remove bogus clearing of the response when xhr.overrideMimeType throws

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/565222693850
part 1.  Add some test coverage for setting xhr.responseType on workers.  r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/2e2e905fdb24
part 2.  Add some test coverage for calling xhr.overrideMimeType() in done state in workers.  r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/7f4dd832985c
part 3.  Remove bogus clearing of the response when xhr.overrideMimeType throws.  r=baku

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/12232 for changes under testing/web-platform/tests

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/565222693850
https://hg.mozilla.org/mozilla-central/rev/2e2e905fdb24
https://hg.mozilla.org/mozilla-central/rev/7f4dd832985c

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged