Matt Brubeck (:mbrubeck) Reporter
	Description • 9 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

	Aaron Raimist [:aaronraimist] Assignee
	Comment 1 • 9 years ago

Bug 1188539 - Remove the deprecated TouchList::identifiedTouch method; r?mbrubeck

	Matt Brubeck (:mbrubeck) Reporter
	Comment 2 • 9 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.

	Aaron Raimist [:aaronraimist] Assignee
	Comment 3 • 9 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

	Aaron Raimist [:aaronraimist] Assignee
	Comment 4 • 9 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.

	Kohei Yoshino [:kohei]
	Comment 5 • 9 years ago

Posted the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/touchlist-identifiedtouch-will-be-removed/

	Matt Brubeck (:mbrubeck) Reporter
	Comment 6 • 8 years ago

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

	Carsten Book [:Tomcat]
	Comment 7 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/ab752670d22a

	Kohei Yoshino [:kohei]
	Comment 8 • 8 years ago

Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2016/touchlist-identifiedtouch-has-been-removed/

	Eric Shepherd [:sheppy]
	Comment 9 • 8 years ago

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.