Remove nsIDOMTouchList

RESOLVED FIXED in mozilla26

Status

()

Core
DOM
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Tracking

unspecified
mozilla26
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 781172 [details] [diff] [review]
Patch
Assignee: nobody → dzbarsky
Status: NEW → ASSIGNED
Attachment #781172 - Flags: review?(reuben.bmo)
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.
Attachment #781172 - Flags: review?(reuben.bmo) → review?(bugs)

Comment 3

4 years ago
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.
Attachment #781172 - Flags: review?(bugs) → review-
(Assignee)

Updated

4 years ago
Depends on: 898930
(Assignee)

Comment 4

4 years ago
Comment on attachment 781172 [details] [diff] [review]
Patch

Given that we're also removing nsIDOMTouchEvent.
Attachment #781172 - Flags: review- → review?(bugs)

Comment 5

4 years ago
Comment on attachment 781172 [details] [diff] [review]
Patch

Assuming nsIDOMTouchEvent goes away in the same hg push, r+.
Attachment #781172 - Flags: review?(bugs) → review+
(Assignee)

Comment 6

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f10e28d82786
(Assignee)

Comment 7

4 years ago
backed out https://hg.mozilla.org/integration/mozilla-inbound/rev/9726efc4c4a8
(Assignee)

Comment 8

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d899f457692
https://hg.mozilla.org/mozilla-central/rev/4d899f457692
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.