Web Animations CSS: Add use counter analytics to CSSAnimations.

Add use counter analytics to CSSAnimations.

Count each specified CSS Transition every time it runs.

Count each requested CSS Animation, including those on non-animatable properties.

BUG=317604
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=161800

[rjwright1, 2013-11-04 08:21:43]

Not for landing, I need to land codereview.chromium.org/57593002 first.

I'm not sure if I've put the use counter in the best spot. If I have, shall I
put it in with splitting keyframes into groups (directly below)?

PTAL.

[rjwright, 2013-11-05 06:36:04]

I made the changes you asked for, Doug. Checked chrome://histograms after
animating left (animatable) and border-style (not animatable) and it picked up
both. I can test further if you want.

[dstockwell, 2013-11-05 22:54:18]

lgtm

Need to link to a bug, and add the 'Web Animations CSS: ' prefix to the change
description.

You should probably file a new bug so that we can close it after we verify that
numbers appear once the flag is flipped and this is deployed in canary.

https://codereview.chromium.org/57683002/diff/80001/Source/core/animation/css...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/57683002/diff/80001/Source/core/animation/css...
Source/core/animation/css/CSSAnimations.cpp:347: // Update use counter.
This comment is not necessary.

[Timothy Loh, 2013-11-10 23:46:39]

You probably shouldn't remove everyone from the cc list...

[dstockwell, 2013-11-10 23:49:20]

On 2013/11/10 23:46:39, Timothy Loh wrote:
> You probably shouldn't remove everyone from the cc list...

/sub

[rjwright1, 2013-11-11 12:18:24]

https://codereview.chromium.org/57683002/diff/80001/Source/core/animation/css...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/57683002/diff/80001/Source/core/animation/css...
Source/core/animation/css/CSSAnimations.cpp:347: // Update use counter.
On 2013/11/05 22:54:18, dstockwell wrote:
> This comment is not necessary.

Done.

[commit-bot: I haz the power, 2013-11-11 12:20:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjwright@chromium.org/57683002/140001

[commit-bot: I haz the power, 2013-11-12 11:38:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjwright@chromium.org/57683002/140001

[commit-bot: I haz the power, 2013-11-12 12:14:53]

Change committed as 161800