Bug 108426

Summary: Use TouchEvent.targetTouches rather than TouchEvent.touches since the order of Touches in TouchList is not guaranteed.
Product: WebKit Reporter: Hayato Ito <hayato>
Component: UI EventsAssignee: Hayato Ito <hayato>
Status: RESOLVED FIXED    
Severity: Normal CC: shinyak, webcomponents-bugzilla, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 107796    
Attachments:
Description Flags
Use targetTouched none

Hayato Ito
Reported 2013-01-30 21:54:14 PST
Neat fix for LayoutTest in http://trac.webkit.org/changeset/141054. In this LayoutTest context, the length of touchEvent.touches is 2. The Touch order in TouchList is not guaranteed. So touchEvent.touches[0] can be another touch. Rather, we should use touchEvent.targetTouches here since its length is 1 in this context.
Attachments
Use targetTouched (1.91 KB, patch)
2013-01-30 21:57 PST, Hayato Ito 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Hayato Ito
Comment 1 2013-01-30 21:57:17 PST
Created attachment 185677 [details] Use targetTouched
Shinya Kawanaka
Comment 2 2013-01-30 22:02:43 PST
Looks OK
WebKit Review Bot
Comment 3 2013-01-31 18:43:00 PST
Comment on attachment 185677 [details] Use targetTouched Clearing flags on attachment: 185677 Committed r141520: <http://trac.webkit.org/changeset/141520>
WebKit Review Bot
Comment 4 2013-01-31 18:43:04 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.