Add UMA measuring pan scrolling usage.

Add UMA measuring pan scrolling usage.

BUG=643167
Committed: https://crrev.com/10f9761b74d51ae9116e16c18fd35bf4713fc819
Cr-Commit-Position: refs/heads/master@{#416991}

[sunyunjia, 2016-09-02 14:22:09]

Patchset #1 (id:1) has been deleted

[sunyunjia, 2016-09-02 14:22:18]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-02 14:22:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 15:58:01]

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

[commit-bot: I haz the power, 2016-09-02 15:58:02]

Dry run: This issue passed the CQ dry run.

[sunyunjia, 2016-09-02 17:42:43]

sunyunjia@chromium.org changed reviewers:
+ bokan@chromium.org, isherman@chromium.org, tdresser@chromium.org

[sunyunjia, 2016-09-02 17:42:44]

PTAL. Thanks!

[bokan, 2016-09-02 19:05:00]

Could we use UseCounter for this? It's easier to test (and we should add a
test).

[Ilya Sherman, 2016-09-02 21:43:31]

On 2016/09/02 19:05:00, bokan wrote:
> Could we use UseCounter for this? It's easier to test (and we should add a
> test).

+1, UseCounter seems more appropriate than a 1-bucket histogram.

[sunyunjia, 2016-09-06 21:36:23]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 21:36:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-06 23:54:05]

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

[commit-bot: I haz the power, 2016-09-06 23:54:06]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sunyunjia, 2016-09-07 00:26:21]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 00:26:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 05:22:43]

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

[commit-bot: I haz the power, 2016-09-07 05:22:44]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sunyunjia, 2016-09-07 05:32:06]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 05:32:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 07:13:25]

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

[commit-bot: I haz the power, 2016-09-07 07:13:26]

Dry run: This issue passed the CQ dry run.

[sunyunjia, 2016-09-07 13:07:57]

PTAL, Thanks!

[tdresser, 2016-09-07 13:41:03]

LGTM with nits.

https://codereview.chromium.org/2302133002/diff/60001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/events/panScroll-use-count.html
(right):

https://codereview.chromium.org/2302133002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/panScroll-use-count.html:13: For
manual testing, hold middle button in scrollable and move aroudn mouse pointer
for scrolling, then release middle button to stop scrolling.
around

https://codereview.chromium.org/2302133002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/panScroll-use-count.html:20: if
(!window.eventSender)
Indentation is wonky.

[sunyunjia, 2016-09-07 13:50:21]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 13:50:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bokan, 2016-09-07 14:31:26]

lgtm

[commit-bot: I haz the power, 2016-09-07 15:35:38]

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

[commit-bot: I haz the power, 2016-09-07 15:35:39]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2016-09-07 17:43:40]

histograms.xml lgtm

[sunyunjia, 2016-09-07 17:49:26]

The CQ bit was checked by sunyunjia@chromium.org

[sunyunjia, 2016-09-07 17:49:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/2302133002/#ps80001
(title: "Fixed the nits")

[commit-bot: I haz the power, 2016-09-07 17:49:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 17:54:15]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-09-07 17:57:51]

Description was changed from

==========
Add UMA measuring pan scrolling usage.

BUG=643167
==========

to

==========
Add UMA measuring pan scrolling usage.

BUG=643167

Committed: https://crrev.com/10f9761b74d51ae9116e16c18fd35bf4713fc819
Cr-Commit-Position: refs/heads/master@{#416991}
==========

[commit-bot: I haz the power, 2016-09-07 17:57:53]

Patchset 4 (id:??) landed as
https://crrev.com/10f9761b74d51ae9116e16c18fd35bf4713fc819
Cr-Commit-Position: refs/heads/master@{#416991}