2008-08-26 Simon Fraser <simon.fraser@apple.com>
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Aug 2008 18:19:55 +0000 (18:19 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Aug 2008 18:19:55 +0000 (18:19 +0000)
        Reviewed by Eric Seidel

        Linear timing functions should have control points 0, 0, 1, 1.
        https://bugs.webkit.org/show_bug.cgi?id=20535

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::mapAnimationTimingFunction):
        * rendering/style/RenderStyle.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@35943 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/transitions/inherit-other-props-expected.txt
LayoutTests/transitions/inherit-other-props.html
WebCore/ChangeLog
WebCore/css/CSSStyleSelector.cpp
WebCore/rendering/style/RenderStyle.h

index f1228a3f5a5295644eff56345c1ee39cc9e90779..01e1365254d7ddb3bc6e9a6c34ee6414a12ad4ef 100644 (file)
@@ -1,3 +1,13 @@
+2008-08-26  Simon Fraser  <simon.fraser@apple.com>
+
+        Reviewed by Eric Seidel
+
+        Linear timing functions should have control points 0, 0, 1, 1.
+        https://bugs.webkit.org/show_bug.cgi?id=20535
+
+        * transitions/inherit-other-props-expected.txt:
+        * transitions/inherit-other-props.html:
+
 2008-08-26  Adam Barth  <abarth@webkit.org>
 
         Reviewed by Darin Adler.
index ec76755fccf73b281066d39a53335e6d2b46d2c6..29d1a5c227efc3ea5ec64a7617a851fc21094421 100644 (file)
@@ -3,16 +3,16 @@ PASS -- Box 1 computed transition duration: 0s expected: 0s
 PASS -- Box 1 computed transition timing function: cubic-bezier(0.25, 0.1, 0.25, 1) expected: cubic-bezier(0.25, 0.1, 0.25, 1)
 PASS -- Box 2 computed transition property: left expected: left
 PASS -- Box 2 computed transition duration: 2s expected: 2s
-PASS -- Box 2 computed transition timing function: cubic-bezier(0, 0, 0, 0) expected: cubic-bezier(0, 0, 0, 0)
+PASS -- Box 2 computed transition timing function: cubic-bezier(0, 0, 1, 1) expected: cubic-bezier(0, 0, 1, 1)
 PASS -- Box 3 computed transition property: left expected: left
 PASS -- Box 3 computed transition duration: 2s expected: 2s
-PASS -- Box 3 computed transition timing function: cubic-bezier(0, 0, 0, 0) expected: cubic-bezier(0, 0, 0, 0)
+PASS -- Box 3 computed transition timing function: cubic-bezier(0, 0, 1, 1) expected: cubic-bezier(0, 0, 1, 1)
 PASS -- Box 4 computed transition property: left expected: left
 PASS -- Box 4 computed transition duration: 2s expected: 2s
-PASS -- Box 4 computed transition timing function: cubic-bezier(0, 0, 0, 0) expected: cubic-bezier(0, 0, 0, 0)
+PASS -- Box 4 computed transition timing function: cubic-bezier(0, 0, 1, 1) expected: cubic-bezier(0, 0, 1, 1)
 PASS -- Box 5 computed transition property: left expected: left
 PASS -- Box 5 computed transition duration: 2s expected: 2s
-PASS -- Box 5 computed transition timing function: cubic-bezier(0, 0, 0, 0) expected: cubic-bezier(0, 0, 0, 0)
+PASS -- Box 5 computed transition timing function: cubic-bezier(0, 0, 1, 1) expected: cubic-bezier(0, 0, 1, 1)
 PASS -- Box 6 computed transition property: all expected: all
 PASS -- Box 6 computed transition duration: 0s expected: 0s
 PASS -- Box 6 computed transition timing function: cubic-bezier(0.25, 0.1, 0.25, 1) expected: cubic-bezier(0.25, 0.1, 0.25, 1)
index 63c7d4c59585d475c141b8bc39699fd2f098222f..a75a5b80743ebe4f266076a84a791b426f52482e 100644 (file)
 
     var kExpectedTimingFunction = [
       'cubic-bezier(0.25, 0.1, 0.25, 1)', /* box1 */
-      'cubic-bezier(0, 0, 0, 0)', /* box2 */
-      'cubic-bezier(0, 0, 0, 0)', /* box3 */
-      'cubic-bezier(0, 0, 0, 0)', /* box4 */ /* inherits from box3 */
-      'cubic-bezier(0, 0, 0, 0)', /* box5 */
+      'cubic-bezier(0, 0, 1, 1)', /* box2 */
+      'cubic-bezier(0, 0, 1, 1)', /* box3 */
+      'cubic-bezier(0, 0, 1, 1)', /* box4 */ /* inherits from box3 */
+      'cubic-bezier(0, 0, 1, 1)', /* box5 */
       'cubic-bezier(0.25, 0.1, 0.25, 1)', /* box6 */ /* does NOT inherit */
     ];
 
index 1fa1681428dc00a248336572a56753af75beb901..1ffc61ff7125f2cf626a9aaa65ae653644dade2b 100644 (file)
@@ -1,3 +1,14 @@
+2008-08-26  Simon Fraser  <simon.fraser@apple.com>
+
+        Reviewed by Eric Seidel
+
+        Linear timing functions should have control points 0, 0, 1, 1.
+        https://bugs.webkit.org/show_bug.cgi?id=20535
+
+        * css/CSSStyleSelector.cpp:
+        (WebCore::CSSStyleSelector::mapAnimationTimingFunction):
+        * rendering/style/RenderStyle.h:
+
 2008-08-27  Adam Roben  <aroben@apple.com>
 
         Windows build fix
index 95e9ef9d02a5fedf4e55267038ce945f46671fe5..5ba81639237287065cad46ccc9178fb98db682c6 100644 (file)
@@ -5239,19 +5239,19 @@ void CSSStyleSelector::mapAnimationTimingFunction(Animation* animation, CSSValue
         CSSPrimitiveValue* primitiveValue = static_cast<CSSPrimitiveValue*>(value);
         switch (primitiveValue->getIdent()) {
             case CSSValueLinear:
-                animation->setTimingFunction(TimingFunction(LinearTimingFunction));
+                animation->setTimingFunction(TimingFunction(LinearTimingFunction, 0.0, 0.0, 1.0, 1.0));
                 break;
             case CSSValueEase:
                 animation->setTimingFunction(TimingFunction());
                 break;
             case CSSValueEaseIn:
-                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, .42, .0, 1.0, 1.0));
+                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, 0.42, 0.0, 1.0, 1.0));
                 break;
             case CSSValueEaseOut:
-                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, .0, .0, .58, 1.0));
+                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, 0.0, 0.0, 0.58, 1.0));
                 break;
             case CSSValueEaseInOut:
-                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, .42, .0, .58, 1.0));
+                animation->setTimingFunction(TimingFunction(CubicBezierTimingFunction, 0.42, 0.0, 0.58, 1.0));
                 break;
         }
         return;
index e0652dd32f6920a79fb1068649328dab36bd55bb..4db62c0f2fd7cc844fcce530d6e4f8ad48de5581 100644 (file)
@@ -1225,13 +1225,13 @@ struct TimingFunction
 {
     TimingFunction()
     : m_type(CubicBezierTimingFunction)
-    , m_x1(.25)
-    , m_y1(.1)
-    , m_x2(.25)
+    , m_x1(0.25)
+    , m_y1(0.1)
+    , m_x2(0.25)
     , m_y2(1.0)
     {}
 
-    TimingFunction(ETimingFunctionType timingFunction, double x1 = .0, double y1 = .0, double x2 = .0, double y2 = .0)
+    TimingFunction(ETimingFunctionType timingFunction, double x1 = 0.0, double y1 = 0.0, double x2 = 1.0, double y2 = 1.0)
     : m_type(timingFunction)
     , m_x1(x1)
     , m_y1(y1)