615507 - Kill DOM L&S with fire

Status: RESOLVED FIXED

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

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

We've never implemented any of L&S, except for, unfortunately, LSProgressEvent. Keeping the idls around could only cause confusion.

Comment 1

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

I filed <http://www.w3.org/Bugs/Public/show_bug.cgi?id=11439> to get the properties on LSProgressEvent specified. Once that's done, we can kill LSProgressEvent too.

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Comment 3

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

This is for after Firefox 4, right?

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Yeah.

Comment 5

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

The one thing we could do for FF4 is to add warnings to progress events when accessing the LSProgressEvent properties. Then maybe we can deprecate them in the future.

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

We could, but should we? I've skimmed over the code, and it doesn't seem like we expose the contents in another way. I'm not a big fan of telling people they can't use a feature if there's nothing they can migrate to. Happy to be proved wrong, though.

Comment 7

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

(In reply to comment #6)
> We could, but should we? I've skimmed over the code, and it doesn't seem like
> we expose the contents in another way. I'm not a big fan of telling people they
> can't use a feature if there's nothing they can migrate to. Happy to be proved
> wrong, though.
We have progress events for XHR, and also for XHR.upload.
Currently during upload the properties of LSProgressEvent report also the header size, but that and the difference in multipart XHR handling are the only differences, IIRC.
IMO the implementation of LSProgressEvent properties is a bit strange,
and should be removed eventually. So we could start to warn about using
them.

Comment 8

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch for checkin

Comment 9

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/projects/cedar/rev/286a6d0c1f40

Comment 10

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/286a6d0c1f40