Increase WebTouchEvent::touchesLengthCap to 16.

Increase WebTouchEvent::touchesLengthCap to 16.

Pixel supports 16 touch-points but Blink caps the number at 12. This is change 2 of 3 on our way to update Blink's limit to 16.

BUG=392959
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183546

[mustaq, 2014-10-07 21:52:10]

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

[mustaq, 2014-10-08 18:10:05]

Will land this after http://crrev.com/639923002.

[Rick Byers, 2014-10-09 14:31:23]

[Sorry for the delay - apparently I never hit publish when I reviewed this
yesterday]

LGTM with nit

https://codereview.chromium.org/637103002/diff/1/public/web/WebInputEvent.h
File public/web/WebInputEvent.h (right):

https://codereview.chromium.org/637103002/diff/1/public/web/WebInputEvent.h#n...
public/web/WebInputEvent.h:488: // FIXME: Make this a variable that somehow
depends on hardware max.
I'm fine saying we'll never plan to support >16 (i.e. omit the FIXME).  There's
not much harm in having this a little bigger than necessary (since events are
not long-lived), and so probably not worth the hassle of making it dynamic.

[mustaq, 2014-10-09 14:53:37]

https://codereview.chromium.org/637103002/diff/1/public/web/WebInputEvent.h
File public/web/WebInputEvent.h (right):

https://codereview.chromium.org/637103002/diff/1/public/web/WebInputEvent.h#n...
public/web/WebInputEvent.h:488: // FIXME: Make this a variable that somehow
depends on hardware max.
On 2014/10/09 14:31:23, Rick Byers wrote:
> I'm fine saying we'll never plan to support >16 (i.e. omit the FIXME). 
There's
> not much harm in having this a little bigger than necessary (since events are
> not long-lived), and so probably not worth the hassle of making it dynamic.

Done.

[mustaq, 2014-10-09 16:50:18]

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

[mustaq, 2014-10-09 16:50:19]

Need owner approval.

[eseidel, 2014-10-10 16:36:51]

The CQ bit was checked by eseidel@chromium.org

[eseidel, 2014-10-10 16:36:52]

lgtm

[eseidel, 2014-10-10 16:37:14]

Rick: you should consider making yourself a public/OWNER.  Just email the
existing owners for approval.

[commit-bot: I haz the power, 2014-10-10 16:37:19]

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

[commit-bot: I haz the power, 2014-10-10 16:40:55]

Committed patchset #2 (id:20001) as 183546