Add touch scrolling modes experimental flags

Add touch scrolling modes experimental flags

In particular, adds --touch-scrolling-mode=absorb-touchmove,
and --touch-scrolling-mode=sync-touchmove.

Split out of https://codereview.chromium.org/156783006, which was migrated from https://codereview.chromium.org/140253005/ so some review history is there.

BUG=328503
TEST=TouchEventQueueTest.SyncTouchMoveDoesntCancelTouchOnScroll,
  TouchEventQueueTest.TouchAbsorption

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251596

[tdresser, 2014-02-14 14:58:17]

joi@, could you take a look at content/public/common/content_switches.*?

Touch scrolling mode specific tests are currently disabled - I'll enable them
once https://codereview.chromium.org/156783006/ lands, and then land this.

[Jói, 2014-02-14 15:05:39]

//content/public LGTM

[Rick Byers, 2014-02-14 20:59:02]

On 2014/02/14 15:05:39, Jói wrote:
> //content/public LGTM

It doesn't look like this piece has changed much from my initial version at
https://codereview.chromium.org/140253005/, so this all LGTM to me.  Since Jared
had review comments there, you should add a link to that original review in the
CL description so someone can follow the review history.

Rick

[jdduke (slow), 2014-02-14 21:17:59]

lgtm with nits

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
File content/browser/renderer_host/input/input_router_impl.cc (right):

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
content/browser/renderer_host/input/input_router_impl.cc:119: {
Nit: Move brace up a line.  Also could you move this function to be by to the
other "configuration" functions above so I don't forget about it when
refactoring?

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
File content/browser/renderer_host/input/touch_event_queue.cc (right):

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
content/browser/renderer_host/input/touch_event_queue.cc:461: // gesture event
(or even part of the current sequence).  Worse case, the
Worse -> Worst

[tdresser, 2014-02-14 21:24:40]

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
File content/browser/renderer_host/input/input_router_impl.cc (right):

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
content/browser/renderer_host/input/input_router_impl.cc:119: {
On 2014/02/14 21:18:00, jdduke wrote:
> Nit: Move brace up a line.  Also could you move this function to be by to the
> other "configuration" functions above so I don't forget about it when
> refactoring?

Done.

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
File content/browser/renderer_host/input/touch_event_queue.cc (right):

https://codereview.chromium.org/166923002/diff/120001/content/browser/rendere...
content/browser/renderer_host/input/touch_event_queue.cc:461: // gesture event
(or even part of the current sequence).  Worse case, the
On 2014/02/14 21:18:00, jdduke wrote:
> Worse -> Worst

Done.

[tdresser, 2014-02-15 13:17:00]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2014-02-15 13:17:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tdresser@chromium.org/166923002/210001

[commit-bot: I haz the power, 2014-02-15 13:37:14]

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

[commit-bot: I haz the power, 2014-02-15 13:37:14]

Retried try job too often on mac_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[tdresser, 2014-02-15 18:30:43]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2014-02-15 18:30:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tdresser@chromium.org/166923002/550001

[commit-bot: I haz the power, 2014-02-15 19:03:37]

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

[commit-bot: I haz the power, 2014-02-15 19:03:38]

Retried try job too often on mac_rel for step(s) app_list_unittests,
base_unittests, browser_tests, content_browsertests, content_unittests,
interactive_ui_tests, ipc_tests, message_center_unittests, nacl_integration,
net_unittests, sql_unittests, sync_integration_tests, sync_unit_tests,
telemetry_perf_unittests, telemetry_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[tdresser, 2014-02-15 19:48:07]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2014-02-15 19:48:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tdresser@chromium.org/166923002/550001

[commit-bot: I haz the power, 2014-02-15 21:11:42]

Change committed as 251596