Add use counter for stacked neutral keyframe in CSS Animations

Add use counter for stacked neutral keyframe in CSS Animations

This use counter tracks whenever a page starts multiple CSS animations
on an element where the tail animations include neutral keyframes.
This is to gauge the occurrence of this in the wild to determine the
viability of altering the behaviour in a subset of these cases.

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

[alancutter (OOO until 2018), 2015-07-21 00:38:05]

alancutter@chromium.org changed reviewers:
+ dstockwell@chromium.org, shans@chromium.org

[alancutter (OOO until 2018), 2015-07-27 01:22:22]

shans dstockwell: Poke.

[shans, 2015-08-06 05:09:30]

https://codereview.chromium.org/1248483005/diff/1/Source/core/animation/css/C...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/1248483005/diff/1/Source/core/animation/css/C...
Source/core/animation/css/CSSAnimations.cpp:266:
createKeyframeEffectModel(resolver, animatingElement, element, &style,
parentStyle, animationName, keyframeTimingFunction.get(), i),
It'd be better to move the UseCounter to these call sites and not have to plumb
animationIndex through.

[alancutter (OOO until 2018), 2015-08-06 07:10:08]

https://codereview.chromium.org/1248483005/diff/1/Source/core/animation/css/C...
File Source/core/animation/css/CSSAnimations.cpp (right):

https://codereview.chromium.org/1248483005/diff/1/Source/core/animation/css/C...
Source/core/animation/css/CSSAnimations.cpp:266:
createKeyframeEffectModel(resolver, animatingElement, element, &style,
parentStyle, animationName, keyframeTimingFunction.get(), i),
On 2015/08/06 at 05:09:30, shans wrote:
> It'd be better to move the UseCounter to these call sites and not have to
plumb animationIndex through.

Done.

[dstockwell, 2015-08-17 07:42:18]

lgtm

[alancutter (OOO until 2018), 2015-08-17 07:43:27]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2015-08-17 07:43:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1248483005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1248483005/40001

[commit-bot: I haz the power, 2015-08-17 07:45:06]

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

[commit-bot: I haz the power, 2015-08-17 07:45:07]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[alancutter (OOO until 2018), 2015-08-18 01:57:25]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-08-18 01:57:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org
Link to the patchset: https://codereview.chromium.org/1248483005/#ps60001
(title: "Rebased")

[commit-bot: I haz the power, 2015-08-18 01:57:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1248483005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1248483005/60001

[commit-bot: I haz the power, 2015-08-18 04:32:13]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200692