Closed Bug 853162 Opened 8 years ago Closed 6 years ago

Remove XMLHttpRequest sendAsBinary

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: annevk, Assigned: djc)

References

Details

(Keywords: addon-compat, dev-doc-needed, site-compat, Whiteboard: [good first bug])

Attachments

(1 file, 1 obsolete file)

It is proprietary and unneeded as far as I can tell. And per bug 681931 might be leaking?
Seems to be used when people don't want a content-type set. See bug 416178 comment 31
You should get the same behavior per spec if you pass a Blob without a type to send(). But yes, we should warn about this going away first.
I agree. lets warn about this for a release or two and then nuke it. But we should be prepared to put it back temporarily if it's breaking high usage websites.
Keywords: dev-doc-needed
Keywords: site-compat
OS: Mac OS X → All
Hardware: x86 → All
It seems this has been deprecated since Firefox 31. Time to remove it?
Whiteboard: [good first bug]
Attached patch Initial version of patch (obsolete) — Splinter Review
Initial version of a patch, pretty much just grepping through the source and removing stuff that seems related.
Assignee: nobody → dirkjan
Status: NEW → ASSIGNED
Attachment #8561031 - Flags: review?(Ms2ger)
Comment on attachment 8561031 [details] [diff] [review]
Initial version of patch

Asking smaug to review based on comment from Ms2ger.
Attachment #8561031 - Flags: review?(Ms2ger) → review?(bugs)
Comment on attachment 8561031 [details] [diff] [review]
Initial version of patch

Could we land this for FF39, so that ESR38 would still have the warning - 
just in case some intranet site uses the property.

Update uuid of nsIXMLHttpRequest.
Attachment #8561031 - Flags: review?(bugs) → review+
So I should just bump the UUID and then we'll hold off for two more weeks?
Add new patch for landing.
Attachment #8561031 - Attachment is obsolete: true
Attachment #8568993 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0ccbd75a97c7
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.