Remove XMLHttpRequest sendAsBinary

RESOLVED FIXED in Firefox 39

Status

()

RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: annevk, Assigned: djc)

Tracking

({addon-compat, dev-doc-needed, site-compat})

unspecified
mozilla39
addon-compat, dev-doc-needed, site-compat
Points:
---

Firefox Tracking Flags

(firefox39 fixed)

Details

(Whiteboard: [good first bug])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
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
(Reporter)

Comment 2

6 years ago
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.

Updated

5 years ago
Keywords: dev-doc-needed
Keywords: site-compat
OS: Mac OS X → All
Hardware: x86 → All
(Reporter)

Comment 4

4 years ago
It seems this has been deprecated since Firefox 31. Time to remove it?
Whiteboard: [good first bug]
(Assignee)

Comment 5

4 years ago
Created attachment 8561031 [details] [diff] [review]
Initial version of patch

Initial version of a patch, pretty much just grepping through the source and removing stuff that seems related.
Assignee: nobody → dirkjan
Status: NEW → ASSIGNED
(Assignee)

Updated

4 years ago
Attachment #8561031 - Flags: review?(Ms2ger)
(Assignee)

Comment 6

4 years ago
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+
(Assignee)

Comment 8

4 years ago
So I should just bump the UUID and then we'll hold off for two more weeks?
(Assignee)

Comment 10

4 years ago
Created attachment 8568993 [details] [diff] [review]
Patch with updated UUID

Add new patch for landing.
Attachment #8561031 - Attachment is obsolete: true
Attachment #8568993 - Flags: review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0ccbd75a97c7
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Keywords: addon-compat
You need to log in before you can comment on or make changes to this bug.