[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes...
authorgraouts@webkit.org <graouts@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2018 19:39:42 +0000 (19:39 +0000)
committergraouts@webkit.org <graouts@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2018 19:39:42 +0000 (19:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186502
<rdar://problem/41000229>l

Reviewed by Dean Jackson.

LayoutTests/imported/w3c:

Mark WPT progression.

* web-platform-tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument-002-expected.txt:

Source/WebCore:

Ensure we only attempt to convert defined objects or we risk to throw errors in valid situations.

* animation/KeyframeEffectReadOnly.cpp:
(WebCore::processKeyframeLikeObject):

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

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument-002-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/animation/KeyframeEffectReadOnly.cpp

index 568b2f9..51a617c 100644 (file)
@@ -1,3 +1,15 @@
+2018-07-06  Antoine Quint  <graouts@apple.com>
+
+        [Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes-argument-002.html pass reliably
+        https://bugs.webkit.org/show_bug.cgi?id=186502
+        <rdar://problem/41000229>l
+
+        Reviewed by Dean Jackson.
+
+        Mark WPT progression.
+
+        * web-platform-tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument-002-expected.txt:
+
 2018-07-05  Antoine Quint  <graouts@apple.com>
 
         [Web Animations] Make WPT test at interfaces/Animation/finish.html pass reliably
index 8191893..1b254de 100644 (file)
@@ -3,6 +3,6 @@ PASS easing values are parsed correctly when set on a property-indexed keyframe
 PASS easing values are parsed correctly when using a keyframe sequence 
 PASS Invalid easing values are correctly rejected when set on a property-indexed keyframe 
 PASS Invalid easing values are correctly rejected when using a keyframe sequence 
-FAIL Errors from invalid easings on a property-indexed keyframe are thrown after reading all properties assert_equals: All properties were read before throwing the easing error expected 2 but got 1
+PASS Errors from invalid easings on a property-indexed keyframe are thrown after reading all properties 
 PASS Errors from invalid easings on a keyframe sequence are thrown after reading all properties 
 
index eac3a58..99d481b 100644 (file)
@@ -1,3 +1,16 @@
+2018-07-06  Antoine Quint  <graouts@apple.com>
+
+        [Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes-argument-002.html pass reliably
+        https://bugs.webkit.org/show_bug.cgi?id=186502
+        <rdar://problem/41000229>l
+
+        Reviewed by Dean Jackson.
+
+        Ensure we only attempt to convert defined objects or we risk to throw errors in valid situations.
+
+        * animation/KeyframeEffectReadOnly.cpp:
+        (WebCore::processKeyframeLikeObject):
+
 2018-07-05  Antoine Quint  <graouts@apple.com>
 
         [Web Animations] Make WPT test at interfaces/Animation/finish.html pass reliably
index 590b402..3ce2528 100644 (file)
@@ -270,7 +270,7 @@ static inline ExceptionOr<KeyframeEffectReadOnly::KeyframeLikeObject> processKey
         // Values as the only element.
         if (rawValue.isString())
             propertyValues = { rawValue.toWTFString(&state) };
-        else
+        else if (rawValue.isObject())
             propertyValues = convert<IDLSequence<IDLDOMString>>(state, rawValue);
         RETURN_IF_EXCEPTION(scope, Exception { TypeError });