898104 - Remove nsIDOMTouchList

Status: RESOLVED FIXED

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

Comment 1

[David Zbarsky (:dzbarsky)]

Attachment: Patch

Comment 2

[Reuben Morais [:reuben]]

Comment on attachment 781172 [details] [diff] [review]
Patch

Review of attachment 781172 [details] [diff] [review]:
-----------------------------------------------------------------

Are we using nsISupports for cases like this? I'd rather have this done after we get rid of nsIDOMTouchEvent, so I'll forwarding the review to someone who can make the decision.

Comment 3

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

Comment on attachment 781172 [details] [diff] [review]
Patch

Yeah, I think leaving nsIDOMTouchEvent rather broken interface isn't good.
We should remove it and nsIDOMTouchList at the same time.

Comment 4

[David Zbarsky (:dzbarsky)]

Comment on attachment 781172 [details] [diff] [review]
Patch

Given that we're also removing nsIDOMTouchEvent.

Comment 5

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

Comment on attachment 781172 [details] [diff] [review]
Patch

Assuming nsIDOMTouchEvent goes away in the same hg push, r+.

Comment 6

[David Zbarsky (:dzbarsky)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f10e28d82786

Comment 7

[David Zbarsky (:dzbarsky)]

backed out https://hg.mozilla.org/integration/mozilla-inbound/rev/9726efc4c4a8

Comment 8

[David Zbarsky (:dzbarsky)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4d899f457692

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/4d899f457692