SVG Web Animations: Add SVGInterpolationType pipeline

SVG Web Animations: Add SVGInterpolationType pipeline

This change starts the SVG family of InterpolationTypes with SVG
specific interpolation environments. It adds SVGValueInterpolationType
for the mode attribute as an initial implementation.
SVG web animation composite modes are not yet introduced in this patch.

BUG=530436
Committed: https://crrev.com/8baf66d6761c194409c707a66606974894b01b67
Cr-Commit-Position: refs/heads/master@{#356804}

[alancutter (OOO until 2018), 2015-10-21 00:15:28]

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

[dstockwell, 2015-10-28 03:29:16]

lgtm

https://codereview.chromium.org/1414413002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/animation/StringKeyframe.cpp (right):

https://codereview.chromium.org/1414413002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/animation/StringKeyframe.cpp:135: const
Vector<const InterpolationType*>* applicableTypesForProperty(PropertyHandle
property)
As discussed I think we'll want to clean this up to move to a separate file, and
initialize once for all properties.

[alancutter (OOO until 2018), 2015-10-28 03:48:45]

https://codereview.chromium.org/1414413002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/animation/StringKeyframe.cpp (right):

https://codereview.chromium.org/1414413002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/animation/StringKeyframe.cpp:135: const
Vector<const InterpolationType*>* applicableTypesForProperty(PropertyHandle
property)
On 2015/10/28 at 03:29:16, dstockwell wrote:
> As discussed I think we'll want to clean this up to move to a separate file,
and initialize once for all properties.

Added TODOs.

[alancutter (OOO until 2018), 2015-10-29 06:24:00]

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

[alancutter (OOO until 2018), 2015-10-29 06:28:09]

Turns out there was a bug in this implementation where I was using
currentValueBase() which includes the animated attribute value so animations
that used the underlying value would feedback into themselves on the next frame.

Added SVGAnimatedPropertyBase::baseValueBase(), +pdr for SVG change.

[fs, 2015-10-29 08:54:52]

On 2015/10/29 at 06:28:09, alancutter wrote:
> Turns out there was a bug in this implementation where I was using
currentValueBase() which includes the animated attribute value so animations
that used the underlying value would feedback into themselves on the next frame.
> 
> Added SVGAnimatedPropertyBase::baseValueBase(), +pdr for SVG change.

svg/ change LGTM

[alancutter (OOO until 2018), 2015-10-29 08:55:38]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-10-29 08:55:38]

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/1414413002/#ps100001
(title: "baseValueBase")

[commit-bot: I haz the power, 2015-10-29 08:55:51]

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

[alancutter (OOO until 2018), 2015-10-29 08:56:45]

The CQ bit was unchecked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-10-29 08:58:43]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-10-29 08:58:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org, fs@opera.com
Link to the patchset: https://codereview.chromium.org/1414413002/#ps120001
(title: "Move assert to not wrong location")

[commit-bot: I haz the power, 2015-10-29 08:59:18]

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

[commit-bot: I haz the power, 2015-10-29 10:27:58]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-10-29 10:28:49]

Patchset 7 (id:??) landed as
https://crrev.com/8baf66d6761c194409c707a66606974894b01b67
Cr-Commit-Position: refs/heads/master@{#356804}