Bug 1369614 - Add test fail condition to reftest of interpolating stroke-width between px and percentage. r?hiro draft
authorMantaroh Yoshinaga <mantaroh@gmail.com>
Fri, 18 Aug 2017 10:32:34 +0900
changeset 648614 17b764a1f0e5457f844a058aa584492d49073eea
parent 648573 a6a1f5c1d971dbee67ba6eec7ead7902351ddca2
child 726875 4760390a80bd981f326cd32cb022cd189d9b0ef3
push id74812
push userbmo:mantaroh@gmail.com
push dateFri, 18 Aug 2017 01:33:32 +0000
reviewershiro
bugs1369614, 1258270, 1386967
milestone57.0a1
Bug 1369614 - Add test fail condition to reftest of interpolating stroke-width between px and percentage. r?hiro Current stylo uses LengthOrPercentage::Calc for interpolation between length and percentage, but neither gecko nor stylo will support the calc value for stroke-dasharray. (bug 1258270 / bug 1386967) So current stylo will not animate this types. In this bug, we make stroke-dasharray interpolation between length and percentage as discrete type until we properly support calc() for stroke-dasharray (bug 1258270 / bug 1386967), as a result, rendering results of this type matches with gecko. MozReview-Commit-ID: FBuJM254ZdS
layout/reftests/svg/smil/style/reftest.list
--- a/layout/reftests/svg/smil/style/reftest.list
+++ b/layout/reftests/svg/smil/style/reftest.list
@@ -109,29 +109,30 @@ skip-if(!styloVsGecko) == anim-css-strok
 == anim-css-strokewidth-1-by-no-no.svg        anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-by-no-no.svg   anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-to-no-no.svg   anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-to-no-no.svg        anim-css-strokewidth-1-ref.svg
 
 # 'stroke-width' property, from/by/to with percent values
 # XXXdholbert the mixed pct + px tests fail right now, because we need calc()
 # in order to interpolate between pct and non-pct values, and we don't yet
-# support calc() for stroke-width & other SVG-specific properties (Bug 594933).
+# support calc() for stroke-width & other SVG-specific properties. (Bug 1258270
+# for gecko, Bug 1386967 for stylo)
 == anim-css-strokewidth-1-by-pct-pct.svg      anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-by-pct-px.svg       anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-by-px-pct.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-by-pct-px.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-by-px-pct.svg       anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-by-pct-pct.svg anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-from-by-pct-px.svg  anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-from-by-px-pct.svg  anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-from-by-pct-px.svg  anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-from-by-px-pct.svg  anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-to-pct-pct.svg anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-from-to-pct-px.svg  anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-from-to-px-pct.svg  anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-from-to-pct-px.svg  anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-from-to-px-pct.svg  anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-to-pct-pct.svg      anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-to-pct-px.svg       anim-css-strokewidth-1-ref.svg
-fails == anim-css-strokewidth-1-to-px-pct.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-to-pct-px.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!styloVsGecko) == anim-css-strokewidth-1-to-px-pct.svg       anim-css-strokewidth-1-ref.svg
 
 # 'stroke-width' property, from/by/to with em values
 == anim-css-strokewidth-1-by-px-em.svg        anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-by-em-em.svg        anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-by-em-px.svg        anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-by-px-em.svg   anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-by-em-em.svg   anim-css-strokewidth-1-ref.svg
 == anim-css-strokewidth-1-from-by-em-px.svg   anim-css-strokewidth-1-ref.svg