Remove XHR.onuploadprogress

RESOLVED FIXED in mozilla17



5 years ago
2 years ago


(Reporter: Ms2ger, Assigned: bz)


({addon-compat, dev-doc-complete})

addon-compat, dev-doc-complete
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags




(1 attachment)

Comment hidden (empty)
Tracking this to make sure we remove again in the next migration (or evaluate needing to apply the patch from 743666)
tracking-firefox15: --- → +
Assigning to bz for now - is there work to be done here now that 15 is on the Beta channel?
Assignee: nobody → bzbarsky
I don't think we need to track this for 15.

Basically, this bug is about removing a nonstandard feature that we've had for a while.  Firefox 14 shipped with a deprecation warning for uses of this feature.  The only question is how long we wait to remove it...

Looking at the addons mxr, I see three uses of onuploadprogress:


We should probably at least get those fixed before we remove it.
We can add it to the compatibility validations for whatever version this ships on, so it isn't necessary to wait for these add-ons to be fixed. We will warn them as long as a I know on which version this is happening.
Keywords: addon-compat
Jorge, thanks!

Peter, any objections to removing this in 17?
Fine by me.
Created attachment 644562 [details] [diff] [review]
No more uploadprogress
Attachment #644562 - Flags: review?(jonas)
Whiteboard: [need review]
Comment on attachment 644562 [details] [diff] [review]
No more uploadprogress

Review of attachment 644562 [details] [diff] [review]:

Attachment #644562 - Flags: review?(jonas) → review+
Flags: in-testsuite-
Whiteboard: [need review]
Backed out due to bustage


for errors
As far as I can tell, this just triggered a rebuild of the generated events stuff somehow and _that_ just failed for whatever reason.  And is continuing to fail even with this patch backed out, since this patch had nothing to do with it to start with...
Relanded as
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17


5 years ago
Keywords: dev-doc-needed


5 years ago
Blocks: 437381
untracking for 15 as per comment 3
tracking-firefox15: + → -
Component: DOM: Mozilla Extensions → DOM
Product: Core → Core
Clearing dev-doc-needed since I didn't think any change was needed in the current XHR doc.
Keywords: dev-doc-needed
Apologies for triggering multiple bugmails.

Updated dev doc:
Keywords: dev-doc-complete
You need to log in before you can comment on or make changes to this bug.