Remove special cases for viewport units, and add tests for viewport units in calc().

Remove special cases for viewport units, and add tests for viewport units in calc().

The check for viewport units in AnimatableLength::interpolateTo should
no longer be necessary now that viewport values are resolved at style
recalc time.

BUG=137617, 168840
TEST=animations/viewport-unit.html
     css3/calc/viewport-unit.html

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

[alancutter (OOO until 2018), 2014-04-15 05:34:07]

lgtm with nits.

The description title should also mention the code removal in this patch.

https://codereview.chromium.org/228063006/diff/1/LayoutTests/animations/viewp...
File LayoutTests/animations/viewport-unit.html (right):

https://codereview.chromium.org/228063006/diff/1/LayoutTests/animations/viewp...
LayoutTests/animations/viewport-unit.html:42:
runAnimationTest(expectedValues(vw));
We prefer not to introduce these types of time dependent layout tests. Please
convert this to an animations/interpolation style test.

[andersr, 2014-04-23 11:10:29]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-04-23 11:10:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/andersr@opera.com/228063006/40001

[commit-bot: I haz the power, 2014-04-23 11:17:24]

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

[commit-bot: I haz the power, 2014-04-23 11:17:25]

Try jobs failed on following builders:
  tryserver.blink on blink_presubmit

[andersr, 2014-04-29 07:40:05]

dstockwell, you should probably review this, as I'm removing a piece of code you
added.

[dstockwell, 2014-04-30 11:00:54]

lgtm

[andersr, 2014-04-30 11:10:47]

The CQ bit was checked by andersr@opera.com

[commit-bot: I haz the power, 2014-04-30 11:10:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/andersr@opera.com/228063006/40001

[commit-bot: I haz the power, 2014-04-30 11:17:13]

Change committed as 172971