Bug 1396491 - Remove the styled border skipped conditions; r?jrmuizel draft
authorKevin Chen <kechen@mozilla.com>
Wed, 25 Oct 2017 10:17:41 +0800
changeset 685946 0137b888886826b23c7898e10315b0ac4db795cb
parent 685608 a124f4901430f6db74cfc7fe3b07957a1c691b40
child 737246 a6b80de01231c7a96b6cdd3158d1aba580a2e9c2
push id86039
push userbmo:kechen@mozilla.com
push dateWed, 25 Oct 2017 06:30:30 +0000
reviewersjrmuizel
bugs1396491
milestone58.0a1
Bug 1396491 - Remove the styled border skipped conditions; r?jrmuizel MozReview-Commit-ID: 4TGeavMJY2Q
layout/generic/crashtests/crashtests.list
layout/painting/nsCSSRenderingBorders.cpp
layout/reftests/border-dotted/reftest.list
layout/reftests/css-break/reftest.list
--- a/layout/generic/crashtests/crashtests.list
+++ b/layout/generic/crashtests/crashtests.list
@@ -640,20 +640,20 @@ asserts(1-4) load 1225005.html # bug 682
 load 1233191.html
 load 1271765.html
 asserts(2) asserts-if(stylo&&Android,1) load 1272983-1.html # bug 586628
 asserts(2) asserts-if(stylo&&Android,1) load 1272983-2.html # bug 586628
 load 1275059.html
 load 1278007.html
 load 1278080.html
 load 1279814.html
-load large-border-radius-dashed.html
-load large-border-radius-dashed2.html
-load large-border-radius-dotted.html
-load large-border-radius-dotted2.html
+skip-if(webrender) load large-border-radius-dashed.html # bug 1409243 not handled by webrender
+skip-if(webrender) load large-border-radius-dashed2.html # bug 1409243 not handled by webrender
+skip-if(webrender) load large-border-radius-dotted.html # bug 1409243 not handled by webrender
+skip-if(webrender) load large-border-radius-dotted2.html # bug 1409243 not handled by webrender
 load 1278461-1.html
 load 1278461-2.html
 load 1281102.html
 load 1297427-non-equal-centers.html
 load 1304441.html
 load 1308876-1.html
 load 1316649.html
 load 1349650.html
--- a/layout/painting/nsCSSRenderingBorders.cpp
+++ b/layout/painting/nsCSSRenderingBorders.cpp
@@ -3586,21 +3586,16 @@ nsCSSBorderRenderer::DrawBorders()
 
 bool
 nsCSSBorderRenderer::CanCreateWebRenderCommands()
 {
   NS_FOR_CSS_SIDES(i) {
     if (mCompositeColors[i] != nullptr) {
       return false;
     }
-
-    if (mBorderStyles[i] == NS_STYLE_BORDER_STYLE_DOTTED ||
-        mBorderStyles[i] == NS_STYLE_BORDER_STYLE_DASHED) {
-      return false;
-    }
   }
 
   return true;
 }
 
 void
 nsCSSBorderRenderer::CreateWebRenderCommands(wr::DisplayListBuilder& aBuilder,
                                              wr::IpcResourceUpdateQueue& aResources,
--- a/layout/reftests/border-dotted/reftest.list
+++ b/layout/reftests/border-dotted/reftest.list
@@ -1,16 +1,16 @@
 # Composing dotted/dashed border rendering and not-filled or not-unfilled mask
 # matches to full-filled box, using fuzzyy because of anti-aliasing.
 # These check filled and unfilled regions with solid opaque color,
 # so difference less than 80 is ignorable.
 
 # XXX: We mark fails-if to these tests for now to enable styled border in WR.
 # We should remove them after Bug 1399485 is fixed.
-fuzzy(80,1500) == border-dotted-interaction.html masked.html
-fuzzy(80,1500) == border-dotted-no-radius.html masked.html
+fuzzy(80,1500) fails-if(webrender) == border-dotted-interaction.html masked.html
+fuzzy(80,1500) fails-if(webrender) == border-dotted-no-radius.html masked.html
 fuzzy(80,1500) fails-if(webrender) == border-dotted-radius.html masked.html
-fuzzy(80,1500) == border-dashed-no-radius.html masked.html
+fuzzy(80,1500) fails-if(webrender) == border-dashed-no-radius.html masked.html
 fuzzy(80,1500) fails-if(webrender) == border-dashed-radius.html masked.html
 fuzzy(80,1500) fails-if(webrender) == border-mixed.html masked.html
 fuzzy(80,1500) fails-if(webrender) == border-dashed-non-integer.html masked.html
 fuzzy(80,1500) fails-if(webrender) == border-dashed-radius-zoom.html masked-zoom.html
-fuzzy(80,1500) == border-dotted-radius-zero.html masked-small.html
+fuzzy(80,1500) fails-if(webrender) == border-dotted-radius-zero.html masked-small.html
--- a/layout/reftests/css-break/reftest.list
+++ b/layout/reftests/css-break/reftest.list
@@ -1,12 +1,12 @@
 default-preferences pref(layout.css.box-decoration-break.enabled,true)
 
 == box-decoration-break-1.html box-decoration-break-1-ref.html
-fuzzy(1,20) fuzzy-if(skiaContent,1,700) fuzzy-if(webrender,17-17,6785-6785) == box-decoration-break-with-inset-box-shadow-1.html box-decoration-break-with-inset-box-shadow-1-ref.html
+fuzzy(1,20) fuzzy-if(skiaContent,1,700) fuzzy-if(webrender,17-17,6741-6741) == box-decoration-break-with-inset-box-shadow-1.html box-decoration-break-with-inset-box-shadow-1-ref.html
 fuzzy(45,460) fuzzy-if(skiaContent,57,439) fuzzy-if(Android,57,1330) fuzzy-if(styloVsGecko,45,1410) == box-decoration-break-with-outset-box-shadow-1.html box-decoration-break-with-outset-box-shadow-1-ref.html # Bug 1386543
 random-if(!gtkWidget) == box-decoration-break-border-image.html box-decoration-break-border-image-ref.html
 == box-decoration-break-block-border-padding.html box-decoration-break-block-border-padding-ref.html
 == box-decoration-break-block-margin.html box-decoration-break-block-margin-ref.html
 fuzzy-if(!Android,1,62) fuzzy-if(Android,8,6627) == box-decoration-break-first-letter.html box-decoration-break-first-letter-ref.html #Bug 1313773
 == box-decoration-break-with-bidi.html box-decoration-break-with-bidi-ref.html
 == box-decoration-break-bug-1235152.html box-decoration-break-bug-1235152-ref.html
 == box-decoration-break-bug-1249913.html box-decoration-break-bug-1249913-ref.html