STR (1) Load http://media.tojicode.com/q3bsp/ (2) Doesn't work, with error "request.overrideMimeType is not a function Source File: http://media.tojicode.com/q3bsp/js/q3bsp_worker.js Line: 70" bent tells me that sicking asked for this to be removed in bug 649537 as part of a larger plan to remove it from the spec. OK, but (i) this breaks some part of the web; and (ii) it's still present in main-thread XHR, against spec. This looks like a difficult battle to me, for what gain?
For what it's worth, I don't think I've asked for overrideMimeType to be removed (I did say that .multipart shouldn't be implemented as it's not part of the spec). So yes, I suspect we need to implement this. I think that we should make it throw if called after .send() is called though since we can't implement it in a non-racy fashion after .send() is called.
(In reply to Chris Jones [:cjones] [:warhammer] from comment #0) > STR > (1) Load http://media.tojicode.com/q3bsp/ FWIW, I added a no-op overrideMimeType() and this page is still broken. Might be separate bustage.
Created attachment 552895 [details] [diff] [review] Patch This adds overrideMimeType back to worker XHR, with the caveat that we'll throw after Send() is called like we discussed on irc. I added a hefty comment explaining the problem. Also, I removed mozResponseArrayBuffer since it shouldn't have been there any more.
Assignee: nobody → bent.mozilla
Status: NEW → ASSIGNED
Attachment #552895 - Flags: review?(jonas)
7 years ago
Summary: overrideMimeType() removed from Workers, breaks web content, diverges from main-thread XHR → Workers: overrideMimeType() removed from XHR, breaks web content, diverges from main-thread XHR
Attachment #552895 - Flags: review?(jonas) → review+
http://hg.mozilla.org/mozilla-central/rev/c5550a4ff624 Could have used a better commit message.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.