Bug 160174

Summary: Touch properties should be on the prototype
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: BindingsAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, benjamin, cdumez, cgarcia, commit-queue, darin, ggaren, mark.lam, rniwa, sam, simon.fraser
Priority: P2 Keywords: WebExposed
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: https://w3c.github.io/touch-events/#idl-def-touch
Attachments:
Description Flags
Patch
none
Patch none

Chris Dumez
Reported 2016-07-25 15:14:47 PDT
Touch properties should be on the prototype: - https://w3c.github.io/touch-events/#idl-def-touch Chrome agrees with the specification.
Attachments
Patch (7.23 KB, patch)
2016-07-25 16:10 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (10.64 KB, patch)
2016-07-25 17:49 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2016-07-25 16:10:44 PDT
Created attachment 284539 [details] Patch
Chris Dumez
Comment 2 2016-07-25 17:49:32 PDT
Created attachment 284555 [details] Patch
WebKit Commit Bot
Comment 3 2016-07-25 19:25:25 PDT
Comment on attachment 284555 [details] Patch Clearing flags on attachment: 284555 Committed r203707: <http://trac.webkit.org/changeset/203707>
WebKit Commit Bot
Comment 4 2016-07-25 19:25:31 PDT
All reviewed patches have been landed. Closing bug.
Simon Fraser (smfr)
Comment 5 2016-08-02 16:21:06 PDT
Can you move the test out of platform/ ? We're trying to avoid new tests there.
Chris Dumez
Comment 6 2016-08-02 16:22:03 PDT
(In reply to comment #5) > Can you move the test out of platform/ ? We're trying to avoid new tests > there. And skip it on other platforms than iOS?
Simon Fraser (smfr)
Comment 7 2016-08-02 16:26:30 PDT
If you use fast/touch/ios it will already be skipped for other platforms.
Simon Fraser (smfr)
Comment 8 2016-08-02 16:26:42 PDT
Or maybe fast/events/touch/ios
Chris Dumez
Comment 9 2016-08-02 16:31:21 PDT
(In reply to comment #8) > Or maybe fast/events/touch/ios Ok, I'll move it shortly.
Chris Dumez
Comment 10 2016-08-03 09:13:57 PDT
(In reply to comment #9) > (In reply to comment #8) > > Or maybe fast/events/touch/ios > > Ok, I'll move it shortly. Moved test in <http://trac.webkit.org/changeset/204081>.
Note You need to log in before you can comment on or make changes to this bug.