Add eventSender support for web pointer properties in touch points.

Add eventSender support for web pointer properties in touch points.

This support is needed for web pointer property propagation layout tests.

This CL is a part of a patch series:
 1.  https://codereview.chromium.org/1253183005/
     for new WebPointerProperties fields
 2.  https://codereview.chromium.org/1260693003/ (this)
 3a. https://codereview.chromium.org/1192563002/
     for handling tilt in Blink event handlers
 3b. https://codereview.chromium.org/1192463008/
     for handling pointer type in Blink event handlers
 4a. https://codereview.chromium.org/1187273004/
     for handling tilt in Chromium event handlers
 4b. https://codereview.chromium.org/1190013002/
     for handling pointer type in Chromium event handlers

BUG=514360
Committed: https://crrev.com/80033921f398c8efe7d751182d2d87f8583e9ba1
Cr-Commit-Position: refs/heads/master@{#341299}

[USE eero AT chromium.org, 2015-07-27 17:59:33]

e.hakkinen@samsung.com changed reviewers:
+ rbyers@chromium.org

[USE eero AT chromium.org, 2015-07-27 17:59:33]

Created based on https://codereview.chromium.org/1192563002/#msg12

I have to create a layout test for checking that this works, does not break
anything and is useful.

[Rick Byers, 2015-07-27 20:57:37]

Seems reasonable to me (the very long list of args may get a little
tedius/brittle, but I don't think we'll need to use it much and can always wrap
it on the JS side with something prettier if we want).

event_sender changes can be a little error-prone (the CQ won't protect you from
breaking the blink build).  So I'll hold my stamp until the blink patch has
landed, and we can run blink tryjobs with this CL.

[mustaq, 2015-07-29 17:11:34]

mustaq@chromium.org changed reviewers:
+ mustaq@chromium.org

[mustaq, 2015-07-29 17:11:34]

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
File components/test_runner/event_sender.cc (right):

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
components/test_runner/event_sender.cc:1758: if (touch_points_[i].id ==
lowest_id)
This is not your code, but seems brittle: could get a duplicate id when
touch_points_[] are not sorted by their ids. Could you please add a TODO here?

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
components/test_runner/event_sender.cc:2382: float* radius_x_pointer,
Setting the fields radiusX & radiusY here appears a bit odd since they are
(still) not in WebPointerProerties. Please add a comment/TODO saying these
fields are here because we expect them to be in WebPointerProperties in future.

[USE eero AT chromium.org, 2015-07-29 17:39:37]

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
File components/test_runner/event_sender.cc (right):

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
components/test_runner/event_sender.cc:1758: if (touch_points_[i].id ==
lowest_id)
On 2015/07/29 17:11:34, mustaq wrote:
> This is not your code, but seems brittle: could get a duplicate id when
> touch_points_[] are not sorted by their ids. Could you please add a TODO here?

Done.

https://codereview.chromium.org/1260693003/diff/1/components/test_runner/even...
components/test_runner/event_sender.cc:2382: float* radius_x_pointer,
On 2015/07/29 17:11:34, mustaq wrote:
> Setting the fields radiusX & radiusY here appears a bit odd since they are
> (still) not in WebPointerProerties. Please add a comment/TODO saying these
> fields are here because we expect them to be in WebPointerProperties in
future.

Done.

[Rick Byers, 2015-07-30 17:44:31]

LGTM.  Thanks for kicking of linux_blink_rel, good to see it's green.  I kicked
off a few others (occasionally we have win/mac-specific event-handler issues or
ASSERTS that only show in Debug).  Assuming there are no failures that are
obviously your fault, go ahead and put this in the CQ.

[USE eero AT chromium.org, 2015-07-31 08:06:52]

The CQ bit was checked by e.hakkinen@samsung.com

[commit-bot: I haz the power, 2015-07-31 08:07:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1260693003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1260693003/20001

[commit-bot: I haz the power, 2015-07-31 08:41:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-31 08:42:40]

Patchset 2 (id:??) landed as
https://crrev.com/80033921f398c8efe7d751182d2d87f8583e9ba1
Cr-Commit-Position: refs/heads/master@{#341299}