Move viewport unit resolution to style recalc time

Source/core/animation/css/CSSAnimatableValueFactory.cpp

Index: Source/core/animation/css/CSSAnimatableValueFactory.cpp
diff --git a/Source/core/animation/css/CSSAnimatableValueFactory.cpp b/Source/core/animation/css/CSSAnimatableValueFactory.cpp
index 1c7a5ca50f63c5cc0a69c76d7c3c06cd7bc1f6d3..15f4615f414b2169edf81b25edd2b520947c175a 100644
--- a/Source/core/animation/css/CSSAnimatableValueFactory.cpp
+++ b/Source/core/animation/css/CSSAnimatableValueFactory.cpp
@@ -68,14 +68,6 @@ static PassRefPtr<AnimatableValue> createFromLength(const Length& length, const
         return AnimatableLength::create(adjustFloatForAbsoluteZoom(length.value(), style), AnimatableLength::UnitTypePixels);
     case Percent:
         return AnimatableLength::create(length.value(), AnimatableLength::UnitTypePercentage);
-    case ViewportPercentageWidth:
-        return AnimatableLength::create(length.value(), AnimatableLength::UnitTypeViewportWidth);
-    case ViewportPercentageHeight:
-        return AnimatableLength::create(length.value(), AnimatableLength::UnitTypeViewportHeight);
-    case ViewportPercentageMin:
-        return AnimatableLength::create(length.value(), AnimatableLength::UnitTypeViewportMin);
-    case ViewportPercentageMax:
-        return AnimatableLength::create(length.value(), AnimatableLength::UnitTypeViewportMax);
     case Calculated:
         return AnimatableLength::create(CSSCalcValue::createExpressionNode(length.calculationValue()->expression(), style.effectiveZoom()));
     case Auto:
@@ -89,6 +81,8 @@ static PassRefPtr<AnimatableValue> createFromLength(const Length& length, const
     case Undefined:
         return AnimatableUnknown::create(CSSValueNone);
     case ExtendToZoom: // Does not apply to elements.
+    case DeviceWidth:

[kenneth.r.christiansen, 2014/01/02 09:03:26]

To avoid anyone abusing these should we call them MetaDeviceWidth ?

[Timothy Loh, 2014/01/03 00:23:09]

Probably doesn't matter too much right now; I want to refactor Length somewhat
in the near-ish future in a way that should make it really obvious if anyone
uses the Length enums wrongly.

+    case DeviceHeight:
         ASSERT_NOT_REACHED();
         return 0;
     }