Bug 260136
| Summary: | [web-animations] linear() probably does not work with transform and other accelerated properties | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Antoine Quint <graouts> |
| Component: | Animations | Assignee: | Antoine Quint <graouts> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | dino, graouts, hi, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=240061 | ||
Antoine Quint
I expect the changes made in 266196@main for bug 240061 are not dealing with accelerated animations. Indeed, CoreAnimation doesn’t have any native equivalent to linear() so we’ll have to either find something that matches and add code to translate to the CA-native primitive or expand KeyframeEffect::canBeAccelerated() to do something similar to what we do with steps(). I argue for the latter because with the work for threaded animation resolution support for linear() will be free and I think it’s not a good idea to add more CA-specific code.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/114185868>
Antoine Quint
<rdar://114084978>
Antoine Quint
Pull request: https://github.com/WebKit/WebKit/pull/19341
EWS
Committed 269594@main (07b0116f3ef9): <https://commits.webkit.org/269594@main>
Reviewed commits have been landed. Closing PR #19341 and removing active labels.