Closed Bug 461066 Opened 13 years ago Closed 10 years ago
Add support for XHR's loadend event
Loadend event has been added to http://dev.w3.org/cvsweb/~checkout~/2006/webapi/progress/Progress.html?rev=1.25
XHR2 should be updated before this, and also HTML 5. I think XHR2/upload should have property onloadend, not sure about <video>/<audio>
<audio> / <video> seem to have a bit different kind of use for progress events and they have more events. So not even sure if loadend is useful there.
I talked to hixie about this. His feeling was that it seemed to make sense for audio/video too given that we have load/abort/error there too.
Summary: Add support for loadend event → Add support for XHR's loadend event
Comment on attachment 519217 [details] [diff] [review] patch Yay!
Attachment #519217 - Flags: review?(jonas) → review+
Actually, could you add this event to the cross-site XHR tests too by adding loadend to the list here: http://mxr.mozilla.org/mozilla-central/source/content/base/test/file_CrossSiteXHR_inner.html?force=1#28 and change the code here to use the loadend event instead: http://mxr.mozilla.org/mozilla-central/source/content/base/test/file_CrossSiteXHR_inner.html?force=1#51 It's unfortunately a bit of a pain since you also need to update file_CrossSiteXHR_inner.jar file_CrossSiteXHR_inner_data.sjs :(
Could you explain that whole CORS testing setup? Why the need for .jar?
I'll push the patch, since it does tests all the parts already, but we certainly should have tests for xsite+loadend too. Filed a followup for that.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Attachment #522046 - Flags: review?(Olli.Pettay) → review+
Checked in CORS tests http://hg.mozilla.org/mozilla-central/rev/2ecbb2d07d85
Documentation updated: https://developer.mozilla.org/en/XPCOM_Interface_Reference/NsIXMLHttpRequestEventTarget https://developer.mozilla.org/En/XMLHttpRequest/Using_XMLHttpRequest#Detecting_any_load_end_condition And listed on Firefox 5 for developers.
You need to log in before you can comment on or make changes to this bug.