Bug 1363880 - Don't skip reftests that don't fail anymore on stylo. r?birtles draft
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Wed, 17 May 2017 05:50:57 +0900
changeset 579072 51be30ea924646511cf9f2de2fe9f4cfacc94140
parent 579032 2cb9844330a3d529b834746eec36784cb1172bad
child 579073 dd1e73b8d4ceea6d8edbae932b0efeaa67e1c7e9
push id59136
push userhikezoe@mozilla.com
push dateTue, 16 May 2017 21:53:59 +0000
reviewersbirtles
bugs1363880
milestone55.0a1
Bug 1363880 - Don't skip reftests that don't fail anymore on stylo. r?birtles MozReview-Commit-ID: 5EHFWdLCd6m
layout/reftests/svg/as-image/reftest.list
layout/reftests/svg/smil/reftest.list
layout/reftests/svg/smil/set/reftest.list
layout/reftests/svg/smil/style/reftest.list
--- a/layout/reftests/svg/as-image/reftest.list
+++ b/layout/reftests/svg/as-image/reftest.list
@@ -61,30 +61,30 @@ skip-if(stylo) test-pref(svg.context-pro
 == context-fill-02.html transparent100x100-w-border-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-02.html lime100x100-w-border-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-03.html lime100x100-50pct-ref.html
 # fuzz because on win8 the r & b components are off by one
 skip-if(stylo) fuzzy-if(winWidget,1,10000) test-pref(svg.context-properties.content.enabled,true) == context-fill-04.html lime100x100-50pct-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-05.html context-fill-or-stroke-05-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-06.html lime100x100-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-07.html context-fill-07-ref.html
-skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-08.html blue100x100-ref.html
+test-pref(svg.context-properties.content.enabled,true) == context-fill-08.html blue100x100-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-fill-bg-image-01.html blue100x100-ref.html
 # context-stroke:
 == context-stroke-01.html blue100x100-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-01.html lime100x100-ref.html
 == context-stroke-02.html transparent100x100-w-border-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-02.html lime100x100-w-border-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-03.html lime100x100-50pct-ref.html
 # fuzz because on win8 the r & b components are off by one
 skip-if(stylo) fuzzy-if(winWidget,1,10000) test-pref(svg.context-properties.content.enabled,true) == context-stroke-04.html lime100x100-50pct-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-05.html context-fill-or-stroke-05-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-06.html lime100x100-ref.html
-skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-07.html context-stroke-07-ref.html
-skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-08.html blue100x100-border-ref.html
+test-pref(svg.context-properties.content.enabled,true) == context-stroke-07.html context-stroke-07-ref.html
+test-pref(svg.context-properties.content.enabled,true) == context-stroke-08.html blue100x100-border-ref.html
 skip-if(stylo) test-pref(svg.context-properties.content.enabled,true) == context-stroke-bg-image-01.html blue100x100-border-ref.html
 
 # Simple <img> tests
 == img-simple-1.html  lime100x100-ref.html
 == img-simple-2.html  lime100x100-ref.html
 fuzzy-if(skiaContent,255,350) == img-simple-3.html  img-simple-3-ref.html
 == img-simple-4.html  lime100x100-ref.html
 fuzzy-if(skiaContent,255,90) == img-simple-5.html  img-simple-5-ref.html
--- a/layout/reftests/svg/smil/reftest.list
+++ b/layout/reftests/svg/smil/reftest.list
@@ -74,21 +74,21 @@ include event/reftest.list
 == anim-indefinite-to-3.svg        anim-standard-ref.svg
 == anim-indefinite-to-4.svg        anim-standard-ref.svg
 
 fails-if(!stylo) == anim-fillcolor-1.svg      anim-standard-ref.svg # bug 436296
 fails-if(stylo) == anim-fillopacity-1none.svg anim-standard-ref.svg
 fails-if(stylo) == anim-fillopacity-1css.svg  anim-standard-ref.svg
 fails-if(stylo) == anim-fillopacity-1xml.svg  anim-standard-ref.svg
 
-fuzzy-if(skiaContent,1,3600) skip-if(stylo) == anim-opacity-01.svg lime.svg
+fuzzy-if(skiaContent,1,3600) == anim-opacity-01.svg lime.svg
 
 == anim-height-done-1a.svg anim-standard-ref.svg
 == anim-height-done-1b.svg anim-standard-ref.svg
-skip-if(stylo) == anim-height-done-2.svg lime.svg
+== anim-height-done-2.svg lime.svg
 == anim-height-interp-1.svg anim-height-interp-1-ref.svg
 == anim-height-interp-2.svg anim-height-interp-2-ref.svg
 == anim-height-interp-3.svg anim-height-interp-3-ref.svg
 == anim-height-interp-4.svg anim-height-interp-4-ref.svg
 == anim-height-interp-5.svg anim-height-interp-5-ref.svg
 == anim-height-interp-6.svg anim-height-interp-6-ref.svg
 
 # check conditional processing
@@ -176,30 +176,30 @@ fails-if(stylo) == anim-additive-fallbac
 
 == anim-remove-1.svg anim-standard-ref.svg
 == anim-remove-2.svg anim-standard-ref.svg
 == anim-remove-3.svg anim-standard-ref.svg
 == anim-remove-4.svg anim-standard-ref.svg
 == anim-remove-5.svg anim-standard-ref.svg
 == anim-remove-6.svg anim-standard-ref.svg
 == anim-remove-7.svg anim-standard-ref.svg
-skip-if(stylo) == anim-remove-8css.svg anim-standard-ref.svg
+== anim-remove-8css.svg anim-standard-ref.svg
 == anim-remove-8xml.svg anim-standard-ref.svg
 fails-if(stylo) == anim-remove-9.svg anim-standard-ref.svg
 == anim-retarget-1.svg anim-standard-ref.svg
 == anim-retarget-2.svg anim-standard-ref.svg
 == anim-retarget-3.svg anim-standard-ref.svg
 == anim-retarget-4.svg anim-standard-ref.svg
 == anim-retarget-5.svg anim-standard-ref.svg
 == anim-retarget-6.svg anim-standard-ref.svg
 == anim-retarget-7.svg anim-standard-ref.svg
 == anim-retarget-8.svg anim-standard-ref.svg
 
 fails-if(!stylo) == anim-strokecolor-1.svg anim-standard-ref.svg # bug 436296
-skip-if(stylo) == anim-strokewidth-1xml.svg anim-standard-ref.svg
+== anim-strokewidth-1xml.svg anim-standard-ref.svg
 
 == anim-targethref-1.svg anim-standard-ref.svg
 == anim-targethref-2.svg anim-standard-ref.svg
 == anim-targethref-3.svg anim-standard-ref.svg
 == anim-targethref-4.svg anim-standard-ref.svg
 == anim-targethref-5.svg anim-standard-ref.svg
 == anim-targethref-6.svg anim-standard-ref.svg
 == anim-targethref-7.svg anim-standard-ref.svg
@@ -257,18 +257,18 @@ fails-if(stylo) == mapped-attr-long-url-
 == min-1.svg lime.svg
 
 fails-if(stylo) == smil-transitions-interaction-1a.svg lime.svg
 fails-if(stylo) == smil-transitions-interaction-1b.svg lime.svg
 fails-if(stylo) == smil-transitions-interaction-2a.svg lime.svg
 fails-if(stylo) == smil-transitions-interaction-2b.svg lime.svg
 == smil-transitions-interaction-3a.svg lime.svg
 == smil-transitions-interaction-3b.svg lime.svg
-skip-if(stylo) == smil-transitions-interaction-4a.svg lime.svg
-skip-if(stylo) == smil-transitions-interaction-4b.svg lime.svg
+== smil-transitions-interaction-4a.svg lime.svg
+== smil-transitions-interaction-4b.svg lime.svg
 
 # Test filtering of excessive times
 == filtered-instance-time-1.svg anim-standard-ref.svg
 
 # Test animation using defs element
 fails-if(stylo) == anim-defs-gradient-property.svg lime.svg
 == anim-defs-gradient-attribute.svg lime.svg
 fails-if(stylo) == anim-defs-fill.svg lime.svg
--- a/layout/reftests/svg/smil/set/reftest.list
+++ b/layout/reftests/svg/smil/set/reftest.list
@@ -1,2 +1,2 @@
-skip-if(stylo) == set-css-fontsize-freeze-1.svg set-css-fontsize-freeze-1-ref.svg
-skip-if(stylo) == set-css-fontsize-remove-1.svg set-css-fontsize-remove-1-ref.svg
+== set-css-fontsize-freeze-1.svg set-css-fontsize-freeze-1-ref.svg
+== set-css-fontsize-remove-1.svg set-css-fontsize-remove-1-ref.svg
--- a/layout/reftests/svg/smil/style/reftest.list
+++ b/layout/reftests/svg/smil/style/reftest.list
@@ -63,93 +63,93 @@ skip-if(stylo) == anim-css-fillopacity-2
 fails == anim-css-fillopacity-3-clamp-big.svg     anim-css-fillopacity-3-ref.svg # bug 501188
 fuzzy-if(skiaContent,1,365) skip-if(stylo) == anim-css-fillopacity-3-clamp-small.svg   anim-css-fillopacity-3-ref.svg
 
 # 'font' shorthand property
 skip-if(stylo) == anim-css-font-1.svg  anim-css-font-1-ref.svg
 
 # 'font-size' property, from/by/to with pixel values only
 skip-if(stylo) == anim-css-fontsize-1-from-by-px-px.svg    anim-css-fontsize-1-ref.svg
-skip-if(stylo) == anim-css-fontsize-1-from-to-px-px.svg    anim-css-fontsize-1-ref.svg
+== anim-css-fontsize-1-from-to-px-px.svg    anim-css-fontsize-1-ref.svg
 
 # 'font-size' property (accepts unitless values)
 skip-if(stylo) == anim-css-fontsize-1-from-to-no-no.svg    anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-css-fontsize-1-from-to-no-px.svg    anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-css-fontsize-1-from-to-px-no.svg    anim-css-fontsize-1-ref.svg
 
 # 'font-size' mapped attribute (accepts unitless values)
 skip-if(stylo) == anim-mapped-fontsize-1-from-to-no-no.svg anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-mapped-fontsize-1-from-to-no-px.svg anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-mapped-fontsize-1-from-to-px-no.svg anim-css-fontsize-1-ref.svg
 
 # 'font-size' property, from/by/to with percent values
 skip-if(stylo) == anim-css-fontsize-1-from-by-pct-pct.svg  anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-css-fontsize-1-from-by-pct-px.svg   anim-css-fontsize-1-ref.svg
 skip-if(stylo) == anim-css-fontsize-1-from-by-px-pct.svg   anim-css-fontsize-1-ref.svg
-skip-if(stylo) == anim-css-fontsize-1-from-to-pct-pct.svg  anim-css-fontsize-1-ref.svg
-skip-if(stylo) == anim-css-fontsize-1-from-to-pct-px.svg   anim-css-fontsize-1-ref.svg
-skip-if(stylo) == anim-css-fontsize-1-from-to-px-pct.svg   anim-css-fontsize-1-ref.svg
+== anim-css-fontsize-1-from-to-pct-pct.svg  anim-css-fontsize-1-ref.svg
+== anim-css-fontsize-1-from-to-pct-px.svg   anim-css-fontsize-1-ref.svg
+== anim-css-fontsize-1-from-to-px-pct.svg   anim-css-fontsize-1-ref.svg
 
 # 'font-size' property, with negative addition
 skip-if(stylo) == anim-css-fontsize-2-from-by-px-px.svg    anim-css-fontsize-2-ref.svg
 skip-if(stylo) == anim-css-fontsize-2-from-by-px-em.svg    anim-css-fontsize-2-ref.svg
 skip-if(stylo) == anim-css-fontsize-2-from-by-em-em.svg    anim-css-fontsize-2-ref.svg
 
 # 'stroke-dasharray' property, from/to with pixel values only
-skip-if(!stylo) skip-if(stylo) == anim-css-strokedasharray-1.svg anim-css-strokedasharray-1-ref.svg
+skip-if(!stylo) == anim-css-strokedasharray-1.svg anim-css-strokedasharray-1-ref.svg
 
 # 'stroke-width' property, from/by/to with pixel values only
 skip-if(stylo) == anim-css-strokewidth-1-by-px-px.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-from-by-px-px.svg   anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-px-px.svg   anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-px-px.svg   anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-to-px-px.svg        anim-css-strokewidth-1-ref.svg
 
 # 'stroke-width' property, from/by/to with unitless values only
 skip-if(stylo) == anim-css-strokewidth-1-by-no-no.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-from-by-no-no.svg   anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-no-no.svg   anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-no-no.svg   anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == 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).
 skip-if(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
 skip-if(stylo) == anim-css-strokewidth-1-from-by-pct-pct.svg anim-css-strokewidth-1-ref.svg
 fails-if(!stylo) == anim-css-strokewidth-1-from-by-pct-px.svg  anim-css-strokewidth-1-ref.svg
 fails-if(!stylo) == anim-css-strokewidth-1-from-by-px-pct.svg  anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-pct-pct.svg anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-pct-pct.svg anim-css-strokewidth-1-ref.svg
 fails-if(!stylo) skip-if(stylo) == anim-css-strokewidth-1-from-to-pct-px.svg  anim-css-strokewidth-1-ref.svg
 fails-if(!stylo) skip-if(stylo) == anim-css-strokewidth-1-from-to-px-pct.svg  anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-to-pct-pct.svg      anim-css-strokewidth-1-ref.svg
-fails-if(!stylo) skip-if(stylo) == anim-css-strokewidth-1-to-pct-px.svg       anim-css-strokewidth-1-ref.svg
-fails-if(!stylo) skip-if(stylo) == anim-css-strokewidth-1-to-px-pct.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!stylo) == anim-css-strokewidth-1-to-pct-px.svg       anim-css-strokewidth-1-ref.svg
+fails-if(!stylo) == anim-css-strokewidth-1-to-px-pct.svg       anim-css-strokewidth-1-ref.svg
 
 # 'stroke-width' property, from/by/to with em values
 skip-if(stylo) == anim-css-strokewidth-1-by-px-em.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-by-em-em.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-by-em-px.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-from-by-px-em.svg   anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-from-by-em-em.svg   anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-from-by-em-px.svg   anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-px-em.svg   anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-em-em.svg   anim-css-strokewidth-1-ref.svg
-skip-if(stylo) == anim-css-strokewidth-1-from-to-em-px.svg   anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-px-em.svg   anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-em-em.svg   anim-css-strokewidth-1-ref.svg
+== anim-css-strokewidth-1-from-to-em-px.svg   anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-to-px-em.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-to-em-em.svg        anim-css-strokewidth-1-ref.svg
 skip-if(stylo) == anim-css-strokewidth-1-to-em-px.svg        anim-css-strokewidth-1-ref.svg
 
 # 'stroke-width' property, values-array & paced calc-mode support
-skip-if(stylo) == anim-css-strokewidth-2-values-em-em.svg    anim-css-strokewidth-2-ref.svg
-skip-if(stylo) == anim-css-strokewidth-2-values-em-px.svg    anim-css-strokewidth-2-ref.svg
-skip-if(stylo) == anim-css-strokewidth-2-values-px-em.svg    anim-css-strokewidth-2-ref.svg
-fuzzy-if(skiaContent,1,360) skip-if(stylo) == anim-css-strokewidth-2-values-px-px.svg    anim-css-strokewidth-2-ref.svg
-skip-if(stylo) == anim-css-strokewidth-3-paced.svg           anim-css-strokewidth-3-ref.svg
+== anim-css-strokewidth-2-values-em-em.svg    anim-css-strokewidth-2-ref.svg
+== anim-css-strokewidth-2-values-em-px.svg    anim-css-strokewidth-2-ref.svg
+== anim-css-strokewidth-2-values-px-em.svg    anim-css-strokewidth-2-ref.svg
+fuzzy-if(skiaContent,1,360) == anim-css-strokewidth-2-values-px-px.svg    anim-css-strokewidth-2-ref.svg
+== anim-css-strokewidth-3-paced.svg           anim-css-strokewidth-3-ref.svg
 
 # 'stroke-width' property, negative addition
 skip-if(stylo) == anim-css-strokewidth-4-from-by-px-px.svg   anim-css-strokewidth-4-ref.svg
 skip-if(stylo) == anim-css-strokewidth-4-from-by-px-em.svg   anim-css-strokewidth-4-ref.svg
 skip-if(stylo) == anim-css-strokewidth-4-from-by-em-em.svg   anim-css-strokewidth-4-ref.svg
 fuzzy-if(skiaContent,1,170) skip-if(stylo) == anim-css-strokewidth-4-from-by-no-no.svg   anim-css-strokewidth-4-ref.svg
 
 # check correct handling of "!important" values