Remove the deprecated TouchList::identifiedTouch method

RESOLVED FIXED in Firefox 49

Status

()

defect
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: mbrubeck, Assigned: aaronraimist, Mentored)

Tracking

({dev-doc-complete, site-compat})

Trunk
mozilla49
Points:
---

Firefox Tracking Flags

(firefox42 affected, firefox49 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
The identifiedTouch method appeared in earlier drafts of the Touch Events v2 specification, but has now been removed from the spec.  It was never implemented by any other browsers:

https://github.com/w3c/touch-events/issues/20

We should remove this interface and its implementation:

https://dxr.mozilla.org/mozilla-central/search?q=identifiedTouch
(Assignee)

Comment 1

4 years ago
Bug 1188539 - Remove the deprecated TouchList::identifiedTouch method; r?mbrubeck
Attachment #8647285 - Flags: review?(mbrubeck)
(Assignee)

Updated

4 years ago
Assignee: nobody → aaronraimist
Status: NEW → ASSIGNED
(Reporter)

Comment 2

4 years ago
Comment on attachment 8647285 [details]
MozReview Request: Bug 1188539 - Remove the deprecated TouchList::identifiedTouch method; r?mbrubeck

Look good!  I'm not officially a reviewer for this code, so I'm requesting a second review from jst.
Attachment #8647285 - Flags: review?(mbrubeck)
Attachment #8647285 - Flags: review?(jst)
Attachment #8647285 - Flags: feedback+
Attachment #8647285 - Flags: review?(jst) → review+
(Assignee)

Comment 3

4 years ago
The try push failed the first time on the mochitest browser chrome tests.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6cd67d69ee89

Ran those tests a second time and now they are green. I don't think this is an issue with my code. This test is being skipped on mozilla-central, see bug 1119906.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6cd67d69ee89
Keywords: checkin-needed
(Assignee)

Comment 4

4 years ago
(In reply to Aaron Raimist [:aaronraimist] from comment #3)
> The try push failed the first time on the mochitest browser chrome tests.
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=6cd67d69ee89
> 
> Ran those tests a second time and now they are green. I don't think this is
> an issue with my code. This test is being skipped on mozilla-central, see
> bug 1119906.
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=6cd67d69ee89

Sorry the second link should have been:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6175cd283bda

but I just realized that this isn't actually running my code, and only ran the try patch (https://hg.mozilla.org/try/rev/6175cd283bda). This might be a bug in Review Board.
Keywords: checkin-needed
(Reporter)

Comment 6

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ab752670d22aae2f4f56cd042a98ec239e0dc931
Bug 1188539 - Remove the deprecated TouchList::identifiedTouch method; r=jst

Comment 7

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ab752670d22a
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Target Milestone: mozilla48 → mozilla49
Someone had already marked this deprecated; I've changed it to obsolete now that it's actually been removed. I also added this function back to the main list on the main TouchList page, since it shouldn't have been removed. It's now correctly there but labeled as obsolete.
You need to log in before you can comment on or make changes to this bug.