RESOLVED FIXED Bug 252925
REGRESSION (STP 164): new failure in css/css-backgrounds/parsing/background-shorthand-serialization.html
https://bugs.webkit.org/show_bug.cgi?id=252925
Summary REGRESSION (STP 164): new failure in css/css-backgrounds/parsing/background-s...
Antoine Quint
Reported 2023-02-24 13:09:32 PST
The last subtest of css/css-backgrounds/parsing/background-shorthand-serialization.html is failing in STP 164 when it used to pass in STP 163: https://wpt.fyi/results/css/css-backgrounds/parsing/background-shorthand-serialization.html?label=master&label=experimental&product=safari-163%20preview&product=safari-164%20preview&view=subtest We need to find out if this is a bug in WebKit or an issue with the test. We should also take the opportunity to import this test so this could be caught with EWS going forward.
Attachments
Darin Adler
Comment 1 2023-02-24 13:32:34 PST
The issue here is "transparent" vs. "none" for the default background which can be specified many ways, including "background: transparent" and "background: none".
Darin Adler
Comment 2 2023-02-24 13:36:17 PST
Darin Adler
Comment 3 2023-02-26 19:53:59 PST
We can just assume that we want this to serialize as "none" and fix it right away rather than waiting for the issues to be resolved. The fix is really easy: 1) Move "background-color" to the end of the list of longhands in CSSProperties.json for "background". 2) Rebase all the tests that change results.
Radar WebKit Bug Importer
Comment 4 2023-03-03 13:10:15 PST
Darin Adler
Comment 5 2023-03-05 14:06:42 PST
EWS
Comment 6 2023-03-05 17:04:07 PST
Committed 261250@main (bec59bdd1e26): <https://commits.webkit.org/261250@main> Reviewed commits have been landed. Closing PR #11087 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.