Bug 125834
| Summary: | Don't depend on layout to do getComputedStyle() on border-radius | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | CSS | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED CONFIGURATION CHANGED | ||
| Severity: | Normal | CC: | ahmad.saleem792, ap, dino, kling, koivisto, mihnea, simon.fraser, zalan |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
Consider merging https://chromium.googlesource.com/chromium/blink/+/2e92d5379e3b3389af543a39e3c476ea20b83933
border-radius and its longhands are defined to return resolved values
instead of used values, this means a layout is not required to compute
the result of getComputedStyle() on these properties.
This patch fixes getComputedStyle() of border radius properties to
return calc objects instead of resolving them to absolute pixel values.
Relevant spec:
http://dev.w3.org/csswg/cssom/#resolved-values
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
Safari passes all border-radius-interpolation tests:
https://wpt.fyi/results/css/css-backgrounds/animations/border-radius-interpolation.html?label=master&label=experimental&aligned&view=subtest&q=border-radius-interpolation
Plus I took other test case and changed it into following JSFiddle:
Link - https://jsfiddle.net/k2w6d3eo/show
^ all browsers (Chrome Canary 106, Firefox Nightly 105 and Safari 15.6 do fail on same test cases) [So they are in consensus]
Another test case - https://jsfiddle.net/tf76v4k5/show
^ all browsers (Chrome Canary 106, Firefox Nightly 105 and Safari 15.6 do fail on same test cases) [So they are in consensus]
____
Is something else required or I am converting these test cases wrong and as a result getting incorrect outputs? Thanks!
Ryosuke Niwa
Sounds like config changed.