242775 2022-07-14 16:25:24 -0700 shorthand serialization for background and mask should not include "initial" in the middle of the value 2022-12-21 07:30:08 -0800 1 1 1 Unclassified WebKit CSS WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=244657 BrowserCompat, InRadar P2 Normal --- 185953 247769 1 heycam darin darin karlcow ntim webkit-bug-importer oldest_to_newest 1883893 0 heycam 2022-07-14 16:25:24 -0700 For example: <!DOCTYPE html> <style> div { background: none; background-position: initial; } </style> <pre id=log></pre> <script> log.textContent = "background: " + document.styleSheets[0].cssRules[0].style.getPropertyValue("background"); </script> This outputs "background: none initial initial", which is not valid. 1885868 1 webkit-bug-importer 2022-07-21 16:26:16 -0700 <rdar://problem/97405629> 1912394 2 darin 2022-11-14 12:11:18 -0800 There is a straightforward way to make sure we don’t serialize any shorthands when one of the longhands has a keyword like "initial". I wrote the code for it and it’s super-simple. But a prerequisite for doing that is that we have to stop setting longhands to "initial" as a side effect of *using* a shorthand. Otherwise, we’ll be getting the empty string way too often. That means we probably need to fix bug 244657 *before* fixing this one. 1912415 3 ntim 2022-11-14 13:32:12 -0800 Does removing `case CSSPropertyBackground:` from the `canUseShorthandForLonghand` blocklist in StyleProperties help? 1913888 4 darin 2022-11-21 10:35:13 -0800 It does not fix this problem, no, but it should be done for other reasons. 1915937 5 darin 2022-12-01 07:51:54 -0800 Pull request: https://github.com/WebKit/WebKit/pull/7025 1917006 6 ews-feeder 2022-12-06 12:55:13 -0800 Committed 257429@main (a0488d1bf07c): <https://commits.webkit.org/257429@main> Reviewed commits have been landed. Closing PR #7025 and removing active labels.