Web Animations: Allow timing inputs outside the range [0, 1]

Source/platform/animation/UnitBezierTest.cpp

Index: Source/platform/animation/UnitBezierTest.cpp
diff --git a/Source/platform/animation/UnitBezierTest.cpp b/Source/platform/animation/UnitBezierTest.cpp
index 85129e8da4726cf6f5ecdc71f4dec8e04100c83e..e8916ae507328e01cec7823779b8e0a1f3820b5f 100644
--- a/Source/platform/animation/UnitBezierTest.cpp
+++ b/Source/platform/animation/UnitBezierTest.cpp
@@ -60,15 +60,29 @@ TEST(UnitBezierTest, InputAtEdgeOfRange)
 TEST(UnitBezierTest, InputOutOfRange)
 {
     UnitBezier bezier(0.5, 1.0, 0.5, 1.0);
-    EXPECT_EQ(0.0, bezier.solve(-1.0, 0.005));
+    EXPECT_EQ(-2.0, bezier.solve(-1.0, 0.005));
     EXPECT_EQ(1.0, bezier.solve(2.0, 0.005));
 }
 
 TEST(UnitBezierTest, InputOutOfRangeLargeEpsilon)
 {
     UnitBezier bezier(0.5, 1.0, 0.5, 1.0);
-    EXPECT_EQ(0.0, bezier.solve(-1.0, 1.0));
+    EXPECT_EQ(-2.0, bezier.solve(-1.0, 1.0));
     EXPECT_EQ(1.0, bezier.solve(2.0, 1.0));
 }
 
+TEST(UnitBezierTest, InputOutOfRangeCoincidentEndpoints)
+{
+    UnitBezier bezier(0.0, 0.0, 1.0, 1.0);
+    EXPECT_EQ(-1.0, bezier.solve(-1.0, 0.005));
+    EXPECT_EQ(2.0, bezier.solve(2.0, 0.005));
+}
+
+TEST(UnitBezierTest, InputOutOfRangeVerticalGradient)
+{
+    UnitBezier bezier(0.0, 1.0, 1.0, 0.0);
+    EXPECT_EQ(0.0, bezier.solve(-1.0, 0.005));
+    EXPECT_EQ(1.0, bezier.solve(2.0, 0.005));
+}

[alancutter (OOO until 2018), 2014/04/16 04:50:42]

Can we add:
TEST(UnitBezierTest, InputOutOfRangeDistinctEndpoints)
{
    UnitBezier bezier(0.1, 0.2, 0.9, 0.7);
    EXPECT_EQ(-2.0, bezier.solve(-1.0, 0.005));
    EXPECT_EQ(4.0, bezier.solve(2.0, 0.005));
}
?

[shans, 2014/04/17 06:04:18]

Done.

+
 } // namespace