Avoid SVG viewport bug for CSS length animations that don't use viewport units

Avoid SVG viewport bug for CSS length animations that don't use viewport units

This change optimises LengthInterpolationFunctions::createLength() to skip
processing units whose corresponding values are zero. This avoids the
nanification of Length for the majority of CSS length animations that don't
use viewport units in SVG images.

See crbug.com/650147 for further details on nan viewport units in SVG images.

BUG=650147, 650216
Committed: https://crrev.com/1ad339d4e001d04db7f63a1d0daa3cb17b5bac64
Cr-Commit-Position: refs/heads/master@{#421137}

[alancutter (OOO until 2018), 2016-09-27 04:44:42]

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

[alancutter (OOO until 2018), 2016-09-27 04:44:48]

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

[commit-bot: I haz the power, 2016-09-27 04:45:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Timothy Loh, 2016-09-27 04:46:55]

lgtm

[alancutter (OOO until 2018), 2016-09-27 04:47:18]

The CQ bit was unchecked by alancutter@chromium.org

[alancutter (OOO until 2018), 2016-09-27 04:47:20]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-09-27 04:47:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[alancutter (OOO until 2018), 2016-09-27 04:56:08]

Description was changed from

==========
Avoid SVG viewport bug for CSS length animations that don't use viewport units

This change optimises LengthInterpolationFunctions::createLength() to skip
processing units whose corresponding values are zero. This avoids the
nanification of Length for the majority of CSS length animations that don't
use viewport units in SVG images.

See bug for further details on nan viewport units in SVG images.

BUG=650147
==========

to

==========
Avoid SVG viewport bug for CSS length animations that don't use viewport units

This change optimises LengthInterpolationFunctions::createLength() to skip
processing units whose corresponding values are zero. This avoids the
nanification of Length for the majority of CSS length animations that don't
use viewport units in SVG images.

See crbug.com/650147 for further details on nan viewport units in SVG images.

BUG=650147,650216
==========

[commit-bot: I haz the power, 2016-09-27 07:34:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 07:36:56]

Description was changed from

==========
Avoid SVG viewport bug for CSS length animations that don't use viewport units

This change optimises LengthInterpolationFunctions::createLength() to skip
processing units whose corresponding values are zero. This avoids the
nanification of Length for the majority of CSS length animations that don't
use viewport units in SVG images.

See crbug.com/650147 for further details on nan viewport units in SVG images.

BUG=650147,650216
==========

to

==========
Avoid SVG viewport bug for CSS length animations that don't use viewport units

This change optimises LengthInterpolationFunctions::createLength() to skip
processing units whose corresponding values are zero. This avoids the
nanification of Length for the majority of CSS length animations that don't
use viewport units in SVG images.

See crbug.com/650147 for further details on nan viewport units in SVG images.

BUG=650147,650216

Committed: https://crrev.com/1ad339d4e001d04db7f63a1d0daa3cb17b5bac64
Cr-Commit-Position: refs/heads/master@{#421137}
==========

[commit-bot: I haz the power, 2016-09-27 07:36:57]

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