Bug 1389000 - Remove duplicated layers-free tests. r?jrmuizel draft
authorKartikaya Gupta <kgupta@mozilla.com>
Mon, 25 Sep 2017 10:41:40 -0400
changeset 669914 64600beb4c20221327653b1674cf3c141f38d099
parent 669913 dfc7409ccc9a7b56dd8280e90890011a992f9ac1
child 669915 c722b233f0231a1455eeca4f947005f6349e16c8
push id81466
push userkgupta@mozilla.com
push dateMon, 25 Sep 2017 15:49:23 +0000
reviewersjrmuizel
bugs1389000
milestone58.0a1
Bug 1389000 - Remove duplicated layers-free tests. r?jrmuizel We had previously started running some tests in layers-free mode already, either by setting the default-preferences on the folder to turn on layers-free, or by duplicating an individual reftest annotation so that it ran in both layers-full and layers-free mode. This patch removes these changes since layers-free is now the default and we don't need to run layers-full at all. MozReview-Commit-ID: Gbz0TRerKCq
layout/reftests/async-scrolling/reftest.list
layout/reftests/bugs/reftest.list
layout/reftests/css-animations/reftest.list
layout/reftests/image/reftest.list
layout/reftests/invalidation/reftest.list
layout/reftests/transform-3d/reftest.list
layout/reftests/transform/reftest.list
layout/reftests/w3c-css/submitted/images3/reftest.list
layout/xul/reftest/reftest.list
--- a/layout/reftests/async-scrolling/reftest.list
+++ b/layout/reftests/async-scrolling/reftest.list
@@ -1,13 +1,8 @@
-# this pref is only meaningful if webrender is enabled. it is reset by the
-# default-preferences line at the end of the file, and so only affects the
-# tests in this folder.
-default-preferences pref(gfx.webrender.layers-free,true)
-
 skip-if(!asyncPan) == bg-fixed-1.html bg-fixed-1-ref.html
 skip-if(!asyncPan) == bg-fixed-cover-1.html bg-fixed-cover-1-ref.html
 skip-if(!asyncPan) == bg-fixed-cover-2.html bg-fixed-cover-2-ref.html
 skip-if(!asyncPan) == bg-fixed-cover-3.html bg-fixed-cover-3-ref.html
 skip-if(!asyncPan) == bg-fixed-child.html bg-fixed-child-ref.html
 skip-if(!asyncPan) == bg-fixed-child-clip-1.html bg-fixed-child-clip-ref.html
 skip-if(!asyncPan) == bg-fixed-child-clip-2.html bg-fixed-child-clip-ref.html
 fuzzy(1,246) fuzzy-if(skiaContent,2,170) fuzzy-if(browserIsRemote&&d2d,53,185) skip-if(!asyncPan) == bg-fixed-child-mask.html bg-fixed-child-mask-ref.html
--- a/layout/reftests/bugs/reftest.list
+++ b/layout/reftests/bugs/reftest.list
@@ -1646,23 +1646,21 @@ HTTP(..) == 621253-2-externalFilter.html
 random-if(winWidget) fails-if(webrender) == 621918-1.svg 621918-1-ref.svg # 1-pixel diacritic positioning discrepancy in rotated text (may depend on platform fonts)
 random-if(winWidget) HTTP(..) == 621918-2.svg 621918-2-ref.svg # same 1px issue as above, and HTTP(..) for filters.svg, used to mask antialiasing issues where glyphs touch
 fuzzy-if(d2d,5,1) == 622585-1.html 622585-1-ref.html # bug 789402
 fuzzy-if(Android,8,300) fuzzy-if(skiaContent,1,40000) == 625409-1.html 625409-1-ref.html
 == 627393-1.html about:blank
 fuzzy-if(skiaContent,1,500) == 630835-1.html about:blank
 == 631352-1.html 631352-1-ref.html
 skip-if(!haveTestPlugin) fails-if(Android) fuzzy-if(winWidget&&!layersGPUAccelerated,102,535) fuzzy-if(skiaContent&&!Android,102,11000) HTTP == 632423-1.html 632423-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == 632423-1.html 632423-1-ref.html
 skip-if(Android) random-if(winWidget||OSX==1010) == 632781-verybig.html 632781-ref.html
 == 632781-normalsize.html 632781-ref.html
 fuzzy-if(d2d&&/^Windows\x20NT\x206\.2/.test(http.oscpu),1,559) fuzzy-if(!isDebugBuild&&gtkWidget&&/^Linux\x20i686/.test(http.oscpu),102,140) == 633344-1.html 633344-1-ref.html # bug 1103623, Linux32 from GCC update
 fuzzy-if(skiaContent,1,500) == 634232-1.html 634232-1-ref.html
 fuzzy-if(skiaContent,3,120000) == 635302-1.html 635302-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == 635302-1.html 635302-1-ref.html
 fuzzy(1,68) fuzzy-if(gtkWidget,1,70) fails-if(Android) fuzzy-if(skiaContent&&!Android,1,300) == 635373-1.html 635373-1-ref.html
 random-if(d2d) fails-if(Android) fuzzy-if(winWidget&&!d2d,20,118) fuzzy-if(skiaContent&&!Android,2,550) == 635373-2.html 635373-2-ref.html
 random-if(d2d) fails-if(Android) fuzzy-if(winWidget&&!d2d,20,116) fuzzy-if(skiaContent&&!Android,2,650) == 635373-3.html 635373-3-ref.html
 HTTP(..) == 635639-1.html 635639-1-ref.html
 HTTP(..) == 635639-2.html 635639-2-ref.html
 random == 637597-1.html 637597-1-ref.html # bug 637597 was never really fixed!
 fuzzy-if(Android,8,500) fuzzy-if(webrender,0-1,0-19) == 637852-1.html 637852-1-ref.html
 fuzzy-if(Android,8,500) fuzzy-if(skiaContent,3,1) fuzzy-if(webrender,3,19) == 637852-2.html 637852-2-ref.html
--- a/layout/reftests/css-animations/reftest.list
+++ b/layout/reftests/css-animations/reftest.list
@@ -38,17 +38,16 @@ random-if(webrender) == stacking-context
 == stacking-context-transform-none-in-delay.html stacking-context-animation-ref.html
 == stacking-context-transform-removing-important-in-delay.html stacking-context-animation-ref.html
 fails == background-position-in-delay.html background-position-ref.html # This test fails the reftest-opaque-layer check since animating background-position currently creates an active layer from its delay phse, and reftest-opaque-layer only handles items assigned to PaintedLayers.
 fails-if(webrender) == background-position-after-finish.html background-position-ref.html
 fails == background-position-running.html background-position-ref.html # This test fails the reftest-opaque-layer check since animating background-position currently creates an active layer, and reftest-opaque-layer only handles items assigned to PaintedLayers.
 fails == background-position-important.html background-position-ref.html # This test fails the reftest-opaque-layer check since animating background-position overridden by a non-animated !important style also creates an active layer, and reftest-opaque-layer only handles items that are assigned to PaintedLayers.
 
 == mask-position-after-finish-1a.html mask-anim-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == mask-position-after-finish-1a.html mask-anim-ref.html
 == mask-position-after-finish-1b.html mask-anim-ref.html
 == mask-position-in-delay-1a.html mask-anim-ref.html
 == mask-position-in-delay-1b.html mask-anim-ref.html
 == mask-size-after-finish-1a.html mask-anim-ref.html
 == mask-size-after-finish-1b.html mask-anim-ref.html
 == mask-size-in-delay-1a.html mask-anim-ref.html
 == mask-size-in-delay-1b.html mask-anim-ref.html
 
--- a/layout/reftests/image/reftest.list
+++ b/layout/reftests/image/reftest.list
@@ -1,17 +1,13 @@
 fuzzy-if(Android,8,30) == background-image-zoom-1.html background-image-zoom-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == background-image-zoom-1.html background-image-zoom-1-ref.html
 fails-if(usesRepeatResampling) == background-image-zoom-2.html about:blank
 == image-seam-1a.html image-seam-1-ref.html
 == image-seam-1b.html image-seam-1-ref.html
 fuzzy-if(Android,255,154) == image-seam-2.html  image-seam-2-ref.html  # Bug 1128229
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == image-seam-1a.html image-seam-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == image-seam-1b.html image-seam-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == image-seam-2.html  image-seam-2-ref.html
 == image-zoom-1.html image-zoom-1-ref.html
 == image-zoom-2.html image-zoom-1-ref.html
 == invalid-url-image-1.html invalid-url-image-1-ref.html
 == sync-image-switch-1a.html sync-image-switch-1-ref.html
 == sync-image-switch-1b.html sync-image-switch-1-ref.html
 == sync-image-switch-1c.html sync-image-switch-1-ref.html
 == sync-image-switch-1d.html sync-image-switch-1-ref.html
 
--- a/layout/reftests/invalidation/reftest.list
+++ b/layout/reftests/invalidation/reftest.list
@@ -81,17 +81,16 @@ pref(layers.single-tile.enabled,false) !
 == background-position-2d.html background-position-2-ref.html
 == background-position-2e.html background-position-2-ref.html
 == background-position-2f.html background-position-2-ref.html
 == zero-opacity-animation.html about:blank
 == zero-opacity-text.html about:blank
 == negative-w-component.html negative-w-component-ref.html
 
 == mask-invalidation-1a.html mask-invalidation-1-ref.html
-pref(gfx.webrender.layers-free,true) skip-if(!webrender) == mask-invalidation-1a.html mask-invalidation-1-ref.html
 == mask-invalidation-1b.html mask-invalidation-1-ref.html
 
 fails-if(webrender) == mask-invalidation-2a.html mask-invalidation-2-ref.html
 fails-if(webrender) == mask-invalidation-2b.html mask-invalidation-2-ref.html
 == mask-invalidation-2c.html mask-invalidation-2-ref.html
 == mask-invalidation-2d.html mask-invalidation-2-ref.html
 
 fails-if(webrender) == clip-path-invalidation-1a.html mask-invalidation-2-ref.html
--- a/layout/reftests/transform-3d/reftest.list
+++ b/layout/reftests/transform-3d/reftest.list
@@ -1,13 +1,8 @@
-# this pref is only meaningful if webrender is enabled. it is reset by the
-# default-preferences line at the end of the file, and so only affects the
-# tests in this folder.
-default-preferences pref(gfx.webrender.layers-free,true)
-
 == scalez-1a.html scalez-1-ref.html
 fuzzy-if(webrender,0-1,0-6) == rotatey-1a.html rotatey-1-ref.html
 == rotatex-1a.html rotatex-1-ref.html
 # Check that scaleZ(-1) rotateX(180deg) is the same as rotateY(180deg)
 == scalezrotatex-1.html scalezrotatex-1-ref.html
 # Check that the perspectve() transform function results in some visual changes
 != rotatex-perspective-1a.html rotatex-1-ref.html
 # Check that -moz-perspective results in visual changes to child transformed elements
@@ -90,11 +85,8 @@ fails-if(webrender) == mask-layer-2.html
 fails-if(webrender) == mask-layer-3.html mask-layer-ref.html
 fails-if(webrender) == split-intersect1.html split-intersect1-ref.html
 fuzzy(255,150) fails-if(webrender) == split-intersect2.html split-intersect2-ref.html
 fuzzy(255,100) fails-if(webrender) == split-non-ortho1.html split-non-ortho1-ref.html
 fuzzy-if(winWidget,150,120) == component-alpha-1.html component-alpha-1-ref.html
 == nested-transform-1.html nested-transform-1-ref.html
 == transform-geometry-1.html transform-geometry-1-ref.html
 == intermediate-1.html intermediate-1-ref.html
-
-# reset gfx.webrender.layers-free,true
-default-preferences
--- a/layout/reftests/transform/reftest.list
+++ b/layout/reftests/transform/reftest.list
@@ -1,13 +1,8 @@
-# this pref is only meaningful if webrender is enabled. it is reset by the
-# default-preferences line at the end of the file, and so only affects the
-# tests in this folder.
-default-preferences pref(gfx.webrender.layers-free,true)
-
 # Transforms specifying singular matrices shouldn't display at all.
 # NOTE: Regressions might manifest themselves as reftest timeouts on
 # this test.
 == singular-1a.html about:blank
 # Multiple transforms should act identically to nested divs.
 fuzzy-if(webrender,1,27) == compound-1a.html compound-1-ref.html
 != compound-1a.html compound-1-fail.html
 == dynamic-inherit-1.html dynamic-inherit-1-ref.html
@@ -147,11 +142,8 @@ pref(svg.transform-box.enabled,true) == 
 # Bug 1301500
 == dynamic-add-without-change-cb-1.html dynamic-add-without-change-cb-1-ref.html
 fuzzy-if(d2d,1,5) fuzzy-if(skiaContent,22,180) == table-overflowed-by-animation.html table-overflowed-by-animation-ref.html
 # Bug 1392161. These tests are failed on Gecko (Bug 1396535). The viewport
 # length transform test is also failed on Servo backend.
 fails-if(!stylo) == translate-rounding-1.html translate-rounding-ref.html
 fails-if(!stylo) == translate-rounding-2.html translate-rounding-ref.html
 fails == translate-rounding-3.html translate-rounding-viewport-ref.html
-
-# reset gfx.webrender.layers-free,true
-default-preferences
--- a/layout/reftests/w3c-css/submitted/images3/reftest.list
+++ b/layout/reftests/w3c-css/submitted/images3/reftest.list
@@ -176,26 +176,26 @@ fails-if(webrender) == object-position-p
 fails-if(webrender) == object-position-png-001i.html object-position-png-001-ref.html
 fails-if(webrender) == object-position-png-001o.html object-position-png-001-ref.html
 fails-if(webrender) == object-position-png-001p.html object-position-png-001-ref.html
 fails == object-position-png-002c.html object-position-png-002-ref.html # bug 1105150
 fails-if(webrender) == object-position-png-002e.html object-position-png-002-ref.html
 fails-if(webrender) == object-position-png-002i.html object-position-png-002-ref.html
 fails-if(webrender) == object-position-png-002o.html object-position-png-002-ref.html
 fails-if(webrender) == object-position-png-002p.html object-position-png-002-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001c.html object-position-png-001-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001e.html object-position-png-001-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001i.html object-position-png-001-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001o.html object-position-png-001-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001p.html object-position-png-001-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002c.html object-position-png-002-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002e.html object-position-png-002-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002i.html object-position-png-002-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002o.html object-position-png-002-ref.html
-pref(gfx.webrender.layers-free,true) pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002p.html object-position-png-002-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001c.html object-position-png-001-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001e.html object-position-png-001-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001i.html object-position-png-001-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001o.html object-position-png-001-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-001p.html object-position-png-001-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002c.html object-position-png-002-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002e.html object-position-png-002-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002i.html object-position-png-002-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002o.html object-position-png-002-ref.html
+pref(layers.advanced.background-image,1) skip-if(!webrender) == object-position-png-002p.html object-position-png-002-ref.html
 fails == object-position-svg-001e.html object-position-svg-001-ref.html # bug 1103286
 == object-position-svg-001i.html object-position-svg-001-ref.html
 fails == object-position-svg-001o.html object-position-svg-001-ref.html # bug 1103286
 == object-position-svg-001p.html object-position-svg-001-ref.html
 fails-if(!webrender) == object-position-svg-002e.html object-position-svg-002-ref.html # bug 1103286
 == object-position-svg-002i.html object-position-svg-002-ref.html
 fails-if(!webrender) == object-position-svg-002o.html object-position-svg-002-ref.html # bug 1103286
 == object-position-svg-002p.html object-position-svg-002-ref.html
--- a/layout/xul/reftest/reftest.list
+++ b/layout/xul/reftest/reftest.list
@@ -1,7 +1,6 @@
 fails-if(Android) == textbox-multiline-noresize.xul textbox-multiline-ref.xul # reference is blank on Android (due to no native theme support?)
 != textbox-multiline-resize.xul textbox-multiline-ref.xul
 == popup-explicit-size.xul popup-explicit-size-ref.xul
-random-if(Android) fails-if(webrender) == image-size.xul image-size-ref.xul
-pref(gfx.webrender.layers-free,true) fuzzy-if(webrender,128-128,168-168) skip-if(!webrender) == image-size.xul image-size-ref.xul
+random-if(Android) fuzzy-if(webrender,128-128,168-168) == image-size.xul image-size-ref.xul
 == image-scaling-min-height-1.xul image-scaling-min-height-1-ref.xul
 == textbox-text-transform.xul textbox-text-transform-ref.xul