797847 - Remove nsIDOMArchiveRequest in favor of nsIDOMDOMRequest

Status: RESOLVED WONTFIX

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

Description

[Mounir Lamouri (:mounir)]

nsIDOMArchiveRequest adds the possibility to access the reader that created the request. I feel like this is rarely going to be used in real life and this is adding more complexity to our code. An example, even our dev engagement team shows examples using the referenced ArchiveReader instead of using the attribute from the request:
https://hacks.mozilla.org/2012/10/archiveapi-read-out-archive-file-contents-introducing-bleeding-edge/

Another point in favour of removing: adding that would be easy. Removing because it is barely used is not.

Comment 1

[Andrea Marchesini [:baku]]

Jonas, I need your feedback about this.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I'd prefer to keep this as-is. While I agree that for simple examples simply keeping a reference to the reader is easy. But once your code gets more complex it seems nice that you can get to the reader through the information handed to the eventhandler.

Comment 3

[Andrea Marchesini [:baku]]

can I resolve it as 'WONTFIX' ?

Comment 4

[Mounir Lamouri (:mounir)]

If both of you disagree, yes...

Comment 5

[Andrea Marchesini [:baku]]

Yes, I agree with the Sicking's point of view.