1525705 - Make WPT css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-{003,010-016}.html pass on windows

Status: NEW

(Core :: Graphics: WebRender, defect, P3)

Description

[Kartikaya Gupta (email:[e-mail])]

The following WPT reftests all seem to fail on windows with red showing up where it shouldn't

! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-003.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-010.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-011.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-012.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-013.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-014.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-015.html
! /css/css-shapes/shape-outside/shape-image/gradients/shape-outside-linear-gradient-016.html

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Kats, do you think we should worry about this for 67?

Comment 2

[Kartikaya Gupta (email:[e-mail])]

So the failures still happen but I tried opening these test pages with a Windows nightly build with WR enabled, and they render fine (after I added a @font-face rule to pick up the Ahem font that the test requires). I'm suspecting there's some weird thing going on in the WPT harness that makes the tests fail in automation.

I can follow up with James Graham but given that these render ok locally we probably don't need to worry about it for 67.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Tests are still marked to fail with WebRender enabled:

https://searchfox.org/mozilla-central/search?q=webrender&path=css%2Fcss-shapes%2Fshape-outside%2Fshape-image%2Fgradients%2Fshape-outside-linear-gradient&case=false&regexp=false

Could someone check why that is still the case?

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Note that in your searchfox query above all of the results are for the not webrender case, which is obsolete now. Pretty much any clause that has not webrender can just be removed, I think. (Jeff can confirm)