716491 - Investigate the status code for non-HTTP XHR.

Status: RESOLVED FIXED

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

Description

[Masatoshi Kimura [:emk]]

Currently we returns 0 instead of 200 for successful non-HTTP responses.

Comment 1

[Masatoshi Kimura [:emk]]

Attachment: Test case

blob data
  0    0       Firefox Nightly + bug 702820 patch
  0    0       Opera 11.60
200  Exception Chrome
200  Exception IE 10 PP3

Comment 2

[Henri Sivonen (:hsivonen)]

(In reply to Masatoshi Kimura [:emk] from comment #1)
> 200  Exception Chrome
> 200  Exception IE 10 PP3

Cool. There is precedent. Let's do this.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

IMO it would be strange to get http codes for non-http XHR, but if IE and webkit does that too,
I'm ok with the change.

Comment 4

[Henri Sivonen (:hsivonen)]

annevk pointed to http://dev.w3.org/2006/webapi/FileAPI/#processingModel for blob:, so our current blob: behavior is clearly wrong.

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I'll take this

Comment 8

[Anne (:annevk)]

Under http://fetch.spec.whatwg.org/#basic-fetch this is mostly defined. Feedback welcome. And e.g. when not stated otherwise, a response's status is by default 200.

Comment 9

[Boris Zbarsky [:bzbarsky]]

I see.  That helps, yes.

Comment 11

[Boris Zbarsky [:bzbarsky]]

Attachment: Return a .status of 200 and .statusText of "OK" for successful non-HTTP XHR loads

Comment 12

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4934b1907579

Comment 13

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/4934b1907579

Comment 14

[Florian Bender]

Release Note Request (optional, but appreciated)
[Why is this notable]: possibly breaking code that assumes false behaviour
[Suggested wording]: Non-HTTP(S) XHR now returns correct status code.
[Links (documentation, blog post, etc)]:

Comment 16

[Saurabh Nair [:jsx] (not reading bugmail. ni? if you need me)]

Release note already updated (by lsblakk, I think) - https://developer.mozilla.org/en-US/Firefox/Releases/35#InterfacesAPIsDOM

Removing dev-doc-needed.