Closed
Bug 716491
Opened 12 years ago
Closed 9 years ago
Investigate the status code for non-HTTP XHR.
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: emk, Assigned: bzbarsky)
References
Details
(Keywords: dev-doc-complete)
Attachments
(2 files)
717 bytes,
text/html
|
Details | |
10.29 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
Currently we returns 0 instead of 200 for successful non-HTTP responses.
Reporter | ||
Comment 1•12 years ago
|
||
blob data 0 0 Firefox Nightly + bug 702820 patch 0 0 Opera 11.60 200 Exception Chrome 200 Exception IE 10 PP3
Comment 2•12 years ago
|
||
(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•12 years ago
|
||
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•12 years ago
|
||
annevk pointed to http://dev.w3.org/2006/webapi/FileAPI/#processingModel for blob:, so our current blob: behavior is clearly wrong.
Reporter | ||
Updated•12 years ago
|
I'll take this
Assignee: nobody → jonas
Updated•12 years ago
|
Keywords: dev-doc-needed
Comment 8•9 years ago
|
||
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.
![]() |
Assignee | |
Comment 9•9 years ago
|
||
I see. That helps, yes.
![]() |
Assignee | |
Comment 11•9 years ago
|
||
Attachment #8484303 -
Flags: review?(bugs)
![]() |
Assignee | |
Updated•9 years ago
|
Assignee: jonas → bzbarsky
Status: NEW → ASSIGNED
Updated•9 years ago
|
Attachment #8484303 -
Flags: review?(bugs) → review+
![]() |
Assignee | |
Comment 12•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4934b1907579
Flags: in-testsuite+
Target Milestone: --- → mozilla35
https://hg.mozilla.org/mozilla-central/rev/4934b1907579
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 14•9 years ago
|
||
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)]:
relnote-firefox:
--- → ?
relnote-b2g:
--- → ?
Updated•9 years ago
|
![]() |
||
Updated•9 years ago
|
Flags: needinfo?(bzbarsky)
![]() |
||
Updated•9 years ago
|
Flags: needinfo?(bzbarsky)
Release note already updated (by lsblakk, I think) - https://developer.mozilla.org/en-US/Firefox/Releases/35#InterfacesAPIsDOM Removing dev-doc-needed.
Keywords: dev-doc-needed
Updated•8 years ago
|
Keywords: dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•