Bug 249816 - Fix css/css-typed-om/the-stylepropertymap/properties/scroll-padding.html WPT test
Summary: Fix css/css-typed-om/the-stylepropertymap/properties/scroll-padding.html WPT ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks: 175733
  Show dependency tree / graph
 
Reported: 2022-12-22 15:08 PST by Chris Dumez
Modified: 2022-12-22 17:13 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 Chris Dumez 2022-12-22 15:08:42 PST 
Fix css/css-typed-om/the-stylepropertymap/properties/scroll-padding.html WPT test to match the specification:
- https://w3c.github.io/csswg-drafts/css-scroll-snap/#padding-longhands-logical
- https://w3c.github.io/csswg-drafts/css-scroll-snap/#padding-longhands-physical

The test was expecting a -3.14% as computed value for one of the subtests. However, the specification indicates that negative values are not allowed.
The computed value should thus be clamped to 0%.
Comment 1 Chris Dumez 2022-12-22 15:10:17 PST 
Pull request: https://github.com/WebKit/WebKit/pull/8027
Comment 2 EWS 2022-12-22 17:12:03 PST 
Committed 258273@main (99f516e7d63b): <https://commits.webkit.org/258273@main>

Reviewed commits have been landed. Closing PR #8027 and removing active labels.
Comment 3 Radar WebKit Bug Importer 2022-12-22 17:13:16 PST 
<rdar://problem/103655472>