Bug 108426 - Use TouchEvent.targetTouches rather than TouchEvent.touches since the order of Touches in TouchList is not guaranteed.
Summary: Use TouchEvent.targetTouches rather than TouchEvent.touches since the order o...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Hayato Ito
URL:
Keywords:
Depends on:
Blocks: 107796
  Show dependency tree / graph
 
Reported: 2013-01-30 21:54 PST by Hayato Ito
Modified: 2013-01-31 18:43 PST (History)
3 users (show)

See Also:

Attachments
Use targetTouched (1.91 KB, patch)
2013-01-30 21:57 PST, Hayato Ito 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Hayato Ito 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.
Comment 1 Hayato Ito 2013-01-30 21:57:17 PST 
Created attachment 185677 [details]
Use targetTouched
Comment 2 Shinya Kawanaka 2013-01-30 22:02:43 PST 
Looks OK
Comment 3 WebKit Review Bot 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>
Comment 4 WebKit Review Bot 2013-01-31 18:43:04 PST 
All reviewed patches have been landed.  Closing bug.