Bug 249770 - [@property] Whitespaces around initial-value should be trimmed
Summary: [@property] Whitespaces around initial-value should be trimmed
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Antti Koivisto
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-12-22 06:18 PST by Antti Koivisto
Modified: 2022-12-22 09:19 PST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Antti Koivisto 2022-12-22 06:18:27 PST 
Partially revert https://commits.webkit.org/258215@main. It turns out that WebKit whitespace handling for custom properties (where whitespace is always trimmed) is correct per https://drafts.csswg.org/css-syntax-3/#consume-declaration and this applies to initial-value too.

This is test by imported/w3c/web-platform-tests/css/css-syntax/declarations-trim-whitespace.html which we pass and other engines fail.

Update the @property tests to the spec.
Comment 1 Radar WebKit Bug Importer 2022-12-22 06:19:25 PST 
<rdar://problem/103636334>
Comment 2 Antti Koivisto 2022-12-22 06:29:49 PST 
Pull request: https://github.com/WebKit/WebKit/pull/8003
Comment 3 Antti Koivisto 2022-12-22 07:07:43 PST 
Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/37647
Comment 4 EWS 2022-12-22 09:19:21 PST 
Committed 258245@main (4f0ccda88791): <https://commits.webkit.org/258245@main>

Reviewed commits have been landed. Closing PR #8003 and removing active labels.