Handle more <length> properties in animation

Handle more <length> properties in css/web animation. 

This change adds the following properties to the LengthStyleInterpolation codepath:

'cx', 'cy', 'r', 'rx', 'ry' and 'stroke-dashoffset'

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

[Erik Dahlström (inactive), 2015-03-10 11:29:20]

ed@opera.com changed reviewers:
+ alancutter@chromium.org, ericwilligers@chromium.org

[Eric Willigers, 2015-03-10 11:46:14]

The CQ bit was checked by ericwilligers@chromium.org

[Eric Willigers, 2015-03-10 11:46:14]

lgtm

[commit-bot: I haz the power, 2015-03-10 11:46:43]

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

[commit-bot: I haz the power, 2015-03-10 12:02:15]

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

[commit-bot: I haz the power, 2015-03-10 12:02:16]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[dstockwell, 2015-03-10 21:20:30]

What's the purpose of this change?

[dstockwell, 2015-03-10 21:20:58]

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

[alancutter (OOO until 2018), 2015-03-10 22:00:33]

This change should be adding tests.
See
LayoutTests/animations/interpolation/svg-stroke-dashoffset-interpolation.html
for an example of a CSS/Web Animation test that applies to SVG properties.
I notice the SVG spec allows unitless numbers for these properties, I'm not sure
if that will work here so you'll want to test that too.

[Erik Dahlström (inactive), 2015-03-11 12:54:11]

On 2015/03/10 21:20:30, dstockwell wrote:
> What's the purpose of this change?

In this particular CL what I'm after is consistency with the other <length>
values wrt css/web animations.

[Erik Dahlström (inactive), 2015-03-11 12:58:32]

On 2015/03/10 22:00:33, alancutter wrote:
> This change should be adding tests.
> See
> LayoutTests/animations/interpolation/svg-stroke-dashoffset-interpolation.html
> for an example of a CSS/Web Animation test that applies to SVG properties.
> I notice the SVG spec allows unitless numbers for these properties, I'm not
sure
> if that will work here so you'll want to test that too.

Thanks to your comment (and adding tests) I found and fixed a couple of bugs,
PTAL.

[alancutter (OOO until 2018), 2015-03-11 23:06:57]

lgtm

[dstockwell, 2015-03-11 23:11:19]

On 2015/03/11 at 12:54:11, ed wrote:
> On 2015/03/10 21:20:30, dstockwell wrote:
> > What's the purpose of this change?
> 
> In this particular CL what I'm after is consistency with the other <length>
values wrt css/web animations.

Could you update the description?

[Erik Dahlström (inactive), 2015-03-12 08:14:46]

The CQ bit was checked by ed@opera.com

[Erik Dahlström (inactive), 2015-03-12 08:14:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
ericwilligers@chromium.org
Link to the patchset: https://codereview.chromium.org/991533003/#ps20001 (title:
"stroke-dashoffset setter exists + fixups + add tests")

[commit-bot: I haz the power, 2015-03-12 08:15:04]

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

[commit-bot: I haz the power, 2015-03-12 08:32:21]

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

[commit-bot: I haz the power, 2015-03-12 08:32:22]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[dstockwell, 2015-03-12 08:45:31]

The CQ bit was checked by dstockwell@chromium.org

[dstockwell, 2015-03-12 08:45:32]

lgtm

[commit-bot: I haz the power, 2015-03-12 08:45:38]

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

[commit-bot: I haz the power, 2015-03-12 08:48:58]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191762