Bug 284724
| Summary: | [cssom] Use shortest possible serialization for border-spacing | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | CSS | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | koivisto, ntim, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While locally imported and seeing new failures, I noticed that we fail following:
WPT Test Case: https://wpt.fyi/results/css/css-tables/parsing/border-spacing-computed.html?label=master&label=experimental&aligned&q=border-spacing
WPT Test Case Live Link: http://wpt.live/css/css-tables/parsing/border-spacing-computed.html
It seems that we serialize it with `space` separated while it shouldn't.
WebKit Source: https://searchfox.org/wubkat/rev/72adbb9f99534551a728acb270d4f111da5e9515/Source/WebCore/css/ComputedStyleExtractor.cpp#3609
Just raising to fix it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/141920587>
Alexsander Borges Damaceno
Pull request: https://github.com/WebKit/WebKit/pull/39503
EWS
Committed 289431@main (6b740c4e4507): <https://commits.webkit.org/289431@main>
Reviewed commits have been landed. Closing PR #39503 and removing active labels.