CC: Add fill mode to compositor animations

CC: Add fill mode to compositor animations

BUG=415241
Committed: https://crrev.com/f671117965e68d5b309bd8ea401aeb90023bdb3d
Cr-Commit-Position: refs/heads/master@{#295614}

[samli, 2014-09-18 00:02:31]

samli@chromium.org changed reviewers:
+ ajuma@chromium.org

[dstockwell, 2014-09-18 00:14:29]

dstockwell@chromium.org changed reviewers:
+ dstockwell@chromium.org

[dstockwell, 2014-09-18 00:14:30]

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation.cc
File cc/animation/animation.cc (right):

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation.cc#ne...
cc/animation/animation.cc:162: bool
Animation::NoEffectBeforeAnimation(base::TimeTicks monotonic_time) const {
This method name is a little strange, perhaps just InEffect(time) (and negating
at the call site)

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation_unitt...
File cc/animation/animation_unittest.cc (right):

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation_unitt...
cc/animation/animation_unittest.cc:620: TEST(AnimationTest,
NoEffectBeforeAnimation) {
Test the interaction with playback rate?

[samli, 2014-09-18 00:59:56]

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation.cc
File cc/animation/animation.cc (right):

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation.cc#ne...
cc/animation/animation.cc:162: bool
Animation::NoEffectBeforeAnimation(base::TimeTicks monotonic_time) const {
On 2014/09/18 00:14:29, dstockwell wrote:
> This method name is a little strange, perhaps just InEffect(time) (and
negating
> at the call site)

Done.

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation_unitt...
File cc/animation/animation_unittest.cc (right):

https://codereview.chromium.org/579863004/diff/1/cc/animation/animation_unitt...
cc/animation/animation_unittest.cc:620: TEST(AnimationTest,
NoEffectBeforeAnimation) {
On 2014/09/18 00:14:29, dstockwell wrote:
> Test the interaction with playback rate?

Done.

[ajuma, 2014-09-18 18:53:10]

lgtm

[samli, 2014-09-18 22:14:00]

The CQ bit was checked by samli@chromium.org

[commit-bot: I haz the power, 2014-09-18 22:17:05]

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

[commit-bot: I haz the power, 2014-09-19 00:01:16]

Committed patchset #4 (id:60001) as 3f5f44ae39885ea9b8698cd30967f08f1590b8f1

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

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