Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the main thread."

Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

TBR=holte@chromium.org
BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram

Review-Url: https://codereview.chromium.org/2651883007
Cr-Commit-Position: refs/heads/master@{#445826}
Committed: https://chromium.googlesource.com/chromium/src/+/b4ff2064283bc966a9de8b8c391dc8a3b168233d

[sahel, 2017-01-24 19:04:16]

Patchset #1 (id:1) has been deleted

[sahel, 2017-01-24 19:08:23]

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

[commit-bot: I haz the power, 2017-01-24 19:09:18]

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, 2017-01-24 20:40:05]

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

[commit-bot: I haz the power, 2017-01-24 20:40:06]

Dry run: This issue passed the CQ dry run.

[sahel, 2017-01-24 20:48:43]

sahel@chromium.org changed reviewers:
+ dtapuska@chromium.org, tdresser@chromium.org

[sahel, 2017-01-24 20:48:44]

This cl is identical to https://codereview.chromium.org/2636583002 which got
reverted because of failing unittests in linex_test(dbg).

I fixed the added unittests and confirmed that the tests pass on debug build as
well.

holte@chromium.org: Please review changes in
histograms.xml

dtapuska@chromium.org: Please review changes in 

tdresser@chromium.org: Please review changes in

[sahel, 2017-01-24 21:00:06]

Description was changed from

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram
==========

to

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram
==========

[sahel, 2017-01-24 21:00:06]

sahel@chromium.org changed reviewers:
+ holte@chromium.org

[dtapuska, 2017-01-24 21:01:47]

On 2017/01/24 20:48:44, sahel wrote:
> This cl is identical to https://codereview.chromium.org/2636583002 which got
> reverted because of failing unittests in linex_test(dbg).
> 
> I fixed the added unittests and confirmed that the tests pass on debug build
as
> well.
> 
> mailto:holte@chromium.org: Please review changes in
> histograms.xml
> 
> mailto:dtapuska@chromium.org: Please review changes in 
> 
> mailto:tdresser@chromium.org: Please review changes in

lgtm

[sahel, 2017-01-24 21:02:48]

Description was changed from

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram
==========

to

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

TBR=holte@chromium.org
BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram
==========

[sahel, 2017-01-24 21:05:49]

On 2017/01/24 21:01:47, dtapuska wrote:
> On 2017/01/24 20:48:44, sahel wrote:
> > This cl is identical to https://codereview.chromium.org/2636583002 which got
> > reverted because of failing unittests in linex_test(dbg).
> > 
> > I fixed the added unittests and confirmed that the tests pass on debug build
> as
> > well.
> > 
> > mailto:holte@chromium.org: Please review changes in
> > histograms.xml
> > 
> > mailto:dtapuska@chromium.org: Please review changes in 
> > 
> > mailto:tdresser@chromium.org: Please review changes in
> 
> lgtm

TBRing holte@ since histograms.xml was reviewed before.

[sahel, 2017-01-24 21:05:58]

The CQ bit was checked by sahel@chromium.org

[commit-bot: I haz the power, 2017-01-24 21:06:22]

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, 2017-01-24 21:39:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485291958646530,
"parent_rev": "c2cdf6f411f8c5e8e7232fbf32b4b1eacea48268", "commit_rev":
"b4ff2064283bc966a9de8b8c391dc8a3b168233d"}

[commit-bot: I haz the power, 2017-01-24 21:39:33]

Description was changed from

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

TBR=holte@chromium.org
BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram
==========

to

==========
Re-landing "UMA metrics for fractions of wheel and touch scrolls blocked on the
main thread."

The original review-URL is https://codereview.chromium.org/2636583002
It got reverted because unittests were failing in Linux_Tests (dbg).

I changed the tests to call ScrollEnd before calling ScrollBegin
for the second time.

TBR=holte@chromium.org
BUG=626662, 626663
TEST=InputHandlerProxyTest.GestureScrollingThreadStatusHistogram,
     InputHandlerProxyTest.WheelScrollingThreadStatusHistogram

Review-Url: https://codereview.chromium.org/2651883007
Cr-Commit-Position: refs/heads/master@{#445826}
Committed:
https://chromium.googlesource.com/chromium/src/+/b4ff2064283bc966a9de8b8c391d...
==========

[commit-bot: I haz the power, 2017-01-24 21:39:34]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/b4ff2064283bc966a9de8b8c391d...

[Steven Holte, 2017-01-25 00:49:10]

lgtm