Add TransitionKeyframeEffectModel specialisation

Add TransitionKeyframeEffectModel specialisation

This patch is part of refactoring CSS Transitions to use
CSSInterpolationTypes instead of AnimatableValues:
https://codereview.chromium.org/2680923005

BUG=681424
Review-Url: https://codereview.chromium.org/2651533004
Cr-Commit-Position: refs/heads/master@{#451563}
Committed: https://chromium.googlesource.com/chromium/src/+/8c72872d7955bffc87b46ce9eff180ed5b0beb4d

[alancutter (OOO until 2018), 2017-02-16 04:33:21]

Description was changed from

==========
Add TransitionKeyframeEffectModel specialisation

BUG=681424
==========

to

==========
Add TransitionKeyframeEffectModel specialisation

This patch is part of refactoring CSS Transitions to use
CSSInterpolationTypes instead of AnimatableValues:
https://codereview.chromium.org/2680923005

BUG=681424
==========

[alancutter (OOO until 2018), 2017-02-16 04:33:21]

alancutter@chromium.org changed reviewers:
+ suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-02-17 00:26:48]

The amount of boilerplate here is nasty, but I won't block this refactor on
figuring out a cleaner way to do it. Consider filing a bug or adding a comment
to my WIP doc for simplifying the interpolation code.

lgtm

[alancutter (OOO until 2018), 2017-02-17 00:37:40]

On 2017/02/17 at 00:26:48, suzyh wrote:
> The amount of boilerplate here is nasty, but I won't block this refactor on
figuring out a cleaner way to do it. Consider filing a bug or adding a comment
to my WIP doc for simplifying the interpolation code.
> 
> lgtm

Agreed, I consider cleaning it up as part of crbug.com/654309.
Once upon a time I thought we'd be able to remove the multiple subclasses of KEM
and delete it all together, alas.

A simple cleanup IMO would be to make a macro for all the macro calls & helper
function definitions, I see that done by other classes in Blink like SVGElement.

[alancutter (OOO until 2018), 2017-02-17 00:39:56]

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

[commit-bot: I haz the power, 2017-02-17 00:41:05]

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-02-17 02:44:01]

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

[commit-bot: I haz the power, 2017-02-17 02:44:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[alancutter (OOO until 2018), 2017-02-20 02:42:52]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2017-02-20 02:43:05]

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-02-20 04:28:02]

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

[commit-bot: I haz the power, 2017-02-20 04:28:03]

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_...)

[alancutter (OOO until 2018), 2017-02-20 04:37:45]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2017-02-20 04:38:07]

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-02-20 05:40:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487565465705060,
"parent_rev": "d8359a676ba9f7c6446f5aa2449466ff517efc9c", "commit_rev":
"8c72872d7955bffc87b46ce9eff180ed5b0beb4d"}

[commit-bot: I haz the power, 2017-02-20 05:40:47]

Description was changed from

==========
Add TransitionKeyframeEffectModel specialisation

This patch is part of refactoring CSS Transitions to use
CSSInterpolationTypes instead of AnimatableValues:
https://codereview.chromium.org/2680923005

BUG=681424
==========

to

==========
Add TransitionKeyframeEffectModel specialisation

This patch is part of refactoring CSS Transitions to use
CSSInterpolationTypes instead of AnimatableValues:
https://codereview.chromium.org/2680923005

BUG=681424

Review-Url: https://codereview.chromium.org/2651533004
Cr-Commit-Position: refs/heads/master@{#451563}
Committed:
https://chromium.googlesource.com/chromium/src/+/8c72872d7955bffc87b46ce9eff1...
==========

[commit-bot: I haz the power, 2017-02-20 05:40:48]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8c72872d7955bffc87b46ce9eff1...