Add modifier flags to MotionEvent

Add modifier flags to MotionEvent

Allow event modifier flags to propagate through the touch and/or
gestures derived from a MotionEvent. This allows, for example,
ctrl + tap to mimic ctrl + click. Note that, previously, Aura routed
these modifier flags separately, and Android failed to route them at
all.

BUG=398439, 413335
Committed: https://crrev.com/36f186d9260f10ab4b134987d369d0dd93f298ff
Cr-Commit-Position: refs/heads/master@{#294695}

[jdduke (slow), 2014-09-11 20:03:14]

jdduke@chromium.org changed reviewers:
+ sadrul@chromium.org, tdresser@chromium.org

[jdduke (slow), 2014-09-11 20:03:14]

PTAL, thanks.

sadrul@: content/browser/renderer_host, content/browser/renderer_host/input
tdresser@: ui/events/gesture_detection/, ui/events/gestures

[tdresser, 2014-09-11 20:51:18]

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
File ui/events/gesture_detection/motion_event.h (right):

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/motion_event.h:43: enum Modifiers {
Where are these used?

[jdduke (slow), 2014-09-11 20:51:57]

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
File ui/events/gesture_detection/motion_event.h (right):

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/motion_event.h:43: enum Modifiers {
On 2014/09/11 20:51:18, tdresser wrote:
> Where are these used?

Oops, they're not, I'll remove.

[jdduke (slow), 2014-09-11 20:57:18]

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
File ui/events/gesture_detection/motion_event.h (right):

https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
ui/events/gesture_detection/motion_event.h:43: enum Modifiers {
On 2014/09/11 20:51:57, jdduke wrote:
> On 2014/09/11 20:51:18, tdresser wrote:
> > Where are these used?
> 
> Oops, they're not, I'll remove.

Done.

[tdresser, 2014-09-12 18:49:00]

On 2014/09/11 20:57:18, jdduke wrote:
>
https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
> File ui/events/gesture_detection/motion_event.h (right):
> 
>
https://codereview.chromium.org/567783002/diff/1/ui/events/gesture_detection/...
> ui/events/gesture_detection/motion_event.h:43: enum Modifiers {
> On 2014/09/11 20:51:57, jdduke wrote:
> > On 2014/09/11 20:51:18, tdresser wrote:
> > > Where are these used?
> > 
> > Oops, they're not, I'll remove.
> 
> Done.

LGTM.

[sadrul, 2014-09-12 22:03:26]

lgtm

[jdduke (slow), 2014-09-12 22:05:37]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-09-12 22:09:21]

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

[commit-bot: I haz the power, 2014-09-12 22:35:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-12 22:35:05]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jdduke (slow), 2014-09-12 22:47:09]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-09-12 22:54:10]

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

[commit-bot: I haz the power, 2014-09-12 23:46:07]

Committed patchset #2 (id:20001) as 8b147ad8a16b31a0b8b5af020d05a9f4d9d75776

[commit-bot: I haz the power, 2014-09-12 23:52:39]

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