Add InputRouter perftests for touches and scroll gestures

Add InputRouter perftests for touches and scroll gestures

Add some basic perftests that track how long a given WebTouchEvent or
WebGestureEvent takes to propagate through the InputRouter.  This includes both
the time to send and the time to ack a given event, averaged over a typical
touch swipe or scroll gesture sequence.

BUG=339877
TBR=jam@chromium.org

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

[jdduke (slow), 2014-02-18 18:52:20]

tdresser@: PTAL.  This is a fairly braindead test, but it picks up changes in
the InputRouter quite nicely (including some tweaks I've been making to
ui::LatencyInfo).  I plan to add a few similar such tests but for gesture
detection when we land the new pipeline in ui/events.

[tdresser, 2014-02-18 19:45:59]

LGTM with nits.

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
File content/browser/renderer_host/input/input_router_impl_perftest.cc (right):

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:1: //
Copyright 2013 The Chromium Authors. All rights reserved.
2014

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:113: Gestures
BuildScrollSequence(size_t steps,
Could this use and/or be a part of synthetic_web_input_event_builders.h?

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:309: void
RunTest(const char* test_name,
I'd prefer it if the "RunTest" methods had unique names.
Perhaps naming this one "RunTouchAndGestureScrollTest" or something along those
lines?

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gypi
File content/content_tests.gypi (right):

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gy...
content/content_tests.gypi:878:
'../ui/accessibility/accessibility.gyp:accessibility',
What is this used for?

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gy...
content/content_tests.gypi:881: '../ui/ui.gyp:ui',
Bringing in all of ui just for some vector operations seems a bit of a shame. I
don't know how to weigh the relative uglyness of including ui vs not being able
to use a nice vector class.

Up to you.

[jdduke (slow), 2014-02-18 20:16:10]

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
File content/browser/renderer_host/input/input_router_impl_perftest.cc (right):

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:1: //
Copyright 2013 The Chromium Authors. All rights reserved.
On 2014/02/18 19:46:00, tdresser wrote:
> 2014

Done.

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:113: Gestures
BuildScrollSequence(size_t steps,
On 2014/02/18 19:46:00, tdresser wrote:
> Could this use and/or be a part of synthetic_web_input_event_builders.h?

Yeah, I have a TODO below to that effect, but I'll file a bug and add a comment
here as well.  Ideally we should simply use the SyntheticGestureController with
a custom SyntheticGestureTarget instance.

https://codereview.chromium.org/170913002/diff/30001/content/browser/renderer...
content/browser/renderer_host/input/input_router_impl_perftest.cc:309: void
RunTest(const char* test_name,
On 2014/02/18 19:46:00, tdresser wrote:
> I'd prefer it if the "RunTest" methods had unique names.
> Perhaps naming this one "RunTouchAndGestureScrollTest" or something along
those
> lines?

Done.

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gypi
File content/content_tests.gypi (right):

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gy...
content/content_tests.gypi:878:
'../ui/accessibility/accessibility.gyp:accessibility',
On 2014/02/18 19:46:00, tdresser wrote:
> What is this used for?

Nothing, see below.

https://codereview.chromium.org/170913002/diff/30001/content/content_tests.gy...
content/content_tests.gypi:881: '../ui/ui.gyp:ui',
On 2014/02/18 19:46:00, tdresser wrote:
> Bringing in all of ui just for some vector operations seems a bit of a shame.
I
> don't know how to weigh the relative uglyness of including ui vs not being
able
> to use a nice vector class.
> 
> Up to you.

Oops, it's not necessary, neither is accessibility... I pulled in a bunch of
dependencies also found in content_unittests when I was bumping into some weird
linker errors, but failed to remove them when I found it was a different issue
altogether.

[jdduke (slow), 2014-02-18 20:17:02]

jam@: Review for content_tests.gypi? Thanks.

[jdduke (slow), 2014-02-19 00:24:01]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-02-19 00:25:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/70002

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

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

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

Retried try job too often on linux_chromeos_clang for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[jdduke (slow), 2014-02-19 06:10:37]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-02-19 06:11:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/70003

[commit-bot: I haz the power, 2014-02-19 07:02:31]

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

[commit-bot: I haz the power, 2014-02-19 07:02:32]

Retried try job too often on linux_clang for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[jdduke (slow), 2014-02-19 07:07:48]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-02-19 07:07:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/70003

[commit-bot: I haz the power, 2014-02-19 08:09:59]

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

[commit-bot: I haz the power, 2014-02-19 08:10:00]

Retried try job too often on linux_clang for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[jdduke (slow), 2014-02-19 14:30:55]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2014-02-19 14:31:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/790001

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/790001

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdduke@chromium.org/170913002/790001

[commit-bot: I haz the power, 2014-02-19 20:59:10]

Change committed as 252079