Fix a rare flakiness in svg/animations/svgPreserveAspectRatio-animation-1.html
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Aug 2015 00:43:48 +0000 (00:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Aug 2015 00:43:48 +0000 (00:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=148149

Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2015-08-18
Reviewed by Tim Horton.

Do not sample the SVG animation at the middle of the animation period. At
that time, the SVG animation test script changes the value of the animated
attribute from the from-value to the to-value. It is safer to sample
immediately before and after the middle of the animation period.

* svg/animations/script-tests/svgPreserveAspectRatio-animation-1.js:
* svg/animations/svgPreserveAspectRatio-animation-1-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/svg/animations/script-tests/svgPreserveAspectRatio-animation-1.js
LayoutTests/svg/animations/svgPreserveAspectRatio-animation-1-expected.txt

index 716dccc..7265ee8 100644 (file)
@@ -1,3 +1,18 @@
+2015-08-18  Said Abou-Hallawa  <sabouhallawa@apple.com>
+
+        Fix a rare flakiness in svg/animations/svgPreserveAspectRatio-animation-1.html
+        https://bugs.webkit.org/show_bug.cgi?id=148149
+
+        Reviewed by Tim Horton.
+        
+        Do not sample the SVG animation at the middle of the animation period. At
+        that time, the SVG animation test script changes the value of the animated
+        attribute from the from-value to the to-value. It is safer to sample
+        immediately before and after the middle of the animation period.
+
+        * svg/animations/script-tests/svgPreserveAspectRatio-animation-1.js:
+        * svg/animations/svgPreserveAspectRatio-animation-1-expected.txt:
+
 2015-08-18  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         Search input results and clear icons are misaligned when padding is added
index 02adaef..8bc02ec 100644 (file)
@@ -51,7 +51,8 @@ function executeTest() {
     const expectedValues = [
         // [animationId, time, sampleCallback]
         ["animation", 0.0,   sample1],
-        ["animation", 2.0,   sample2],
+        ["animation", 1.999, sample2],
+        ["animation", 2.001, sample3],
         ["animation", 3.999, sample3],
         ["animation", 4.001, sample1]
     ];
index 9f332ef..79b64bd 100644 (file)
@@ -17,6 +17,10 @@ PASS rootSVGElement.preserveAspectRatio.animVal.align is SVGPreserveAspectRatio.
 PASS rootSVGElement.preserveAspectRatio.animVal.meetOrSlice is SVGPreserveAspectRatio.SVG_MEETORSLICE_SLICE
 PASS rootSVGElement.preserveAspectRatio.baseVal.align is SVGPreserveAspectRatio.SVG_PRESERVEASPECTRATIO_XMAXYMIN
 PASS rootSVGElement.preserveAspectRatio.baseVal.meetOrSlice is SVGPreserveAspectRatio.SVG_MEETORSLICE_MEET
+PASS rootSVGElement.preserveAspectRatio.animVal.align is SVGPreserveAspectRatio.SVG_PRESERVEASPECTRATIO_XMAXYMID
+PASS rootSVGElement.preserveAspectRatio.animVal.meetOrSlice is SVGPreserveAspectRatio.SVG_MEETORSLICE_SLICE
+PASS rootSVGElement.preserveAspectRatio.baseVal.align is SVGPreserveAspectRatio.SVG_PRESERVEASPECTRATIO_XMAXYMIN
+PASS rootSVGElement.preserveAspectRatio.baseVal.meetOrSlice is SVGPreserveAspectRatio.SVG_MEETORSLICE_MEET
 PASS rootSVGElement.preserveAspectRatio.animVal.align is SVGPreserveAspectRatio.SVG_PRESERVEASPECTRATIO_XMAXYMIN
 PASS rootSVGElement.preserveAspectRatio.animVal.meetOrSlice is SVGPreserveAspectRatio.SVG_MEETORSLICE_MEET
 PASS rootSVGElement.preserveAspectRatio.baseVal.align is SVGPreserveAspectRatio.SVG_PRESERVEASPECTRATIO_XMAXYMIN