WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
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
Tim filed this
https://github.com/w3c/csswg-drafts/issues/8496
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
<
rdar://problem/106208628
>
Darin Adler
Comment 5
2023-03-05 14:06:42 PST
Pull request:
https://github.com/WebKit/WebKit/pull/11087
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.
Top of Page
Format For Printing
XML
Clone This Bug