New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip invalid SVG+currentcolor tests. #4508
Skip invalid SVG+currentcolor tests. #4508
Conversation
EWS run on previous version of this PR (hash 93ff1a1) |
93ff1a1
to
c7f8b4c
Compare
# Old SVG+currentcolor tests are incorrect https://bugs.webkit.org/show_bug.cgi?id=245400 | ||
imported/w3c/web-platform-tests/svg/import/color-prop-05-t-manual.svg [ Skip ] | ||
svg/W3C-SVG-1.1-SE/color-prop-05-t.svg [ Skip ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should look into de-duplicating these tests, and fixing it upstream as well. I can take care of that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes indeed, there is bugzilla about those de-duplication tasks https://bugs.webkit.org/show_bug.cgi?id=215135
https://bugs.webkit.org/show_bug.cgi?id=245400 Reviewed by Tim Nguyen. With SVG 2 and Color level 5, it should define a rectangle fill in red (not lime). This is the behaviour of both Gecko and Blink. * LayoutTests/TestExpectations: Canonical link: https://commits.webkit.org/254649@main
c7f8b4c
to
54fc90a
Compare
Committed 254649@main (54fc90a): https://commits.webkit.org/254649@main Reviewed commits have been landed. Closing PR #4508 and removing active labels. |
54fc90a
c7f8b4c
π macπ wpeπ π§ͺ winπ ios-simπ mac-debugπ gtkπ wincairoπ§ͺ ios-wk2π mac-AS-debugπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ api-macπ§ͺ api-gtkπ tvπ§ͺ mac-wk1π tv-simπ§ͺ mac-wk2π watchπ§ͺ mac-AS-debug-wk2π watch-simπ§ͺ mac-wk2-stress