906897 - Calling abort() on an inactive FileReader throws an exception

Status: NEW

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

Description

[Arun Ranganathan]

Current Behavior: 

var f = new FileReader();
f.abort();

will throw an exception, owing to bug 657964.

Expected Behavior:

The File API says: http://dev.w3.org/2006/webapi/FileAPI/#abort which is to do nothing about an abort() call when the status is EMPTY or DONE, NOT to throw an exception.

Other Browsers:

Chrome and Safari do not throw an exception.

Comment 1

[Brendan Eich [:brendan]]

Is this a regression?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Somewhat: we purposefully changed to a behavior that does not match the spec.

Comment 3

[Arun Ranganathan]

bz, I think that this behavior was introduced because the spec. itself was in flux.  I don't think it was a willful violation of spec.  If you think spec. behavior is wrong, I'd like to know of course :)

Comment 4

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

I vaguely recall that we intended to align with XHR or something here.  Maybe that never ended up happening.

Comment 5

[Anne (:annevk)]

XMLHttpRequest's abort() never throws. I think generally we should throwing unless it catches something that's an actual mistake or cannot be readily handled. I don't think that's the case here.

Comment 6

[riadh.chtara]

Attachment: FileReaderException.diff

Comment 7

[riadh.chtara]

Hey guys,
Should I also remove [Throws] just before void abort(); in FileReader.webid?
Thanks

Comment 8

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 826457 [details] [diff] [review]
FileReaderException.diff

Yes, you should remove [Throws] and the ErrorResult param since per spec
the method doesn't throw.

Comment 9

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.