Bug 282750
Summary: | [threaded-animation-resolution] rename `AcceleratedTimeline` to `AcceleratedEffectStackUpdater` and make it accessible via `AnimationTimelinesController` | ||
---|---|---|---|
Product: | WebKit | Reporter: | Antoine Quint <graouts> |
Component: | Animations | Assignee: | Antoine Quint <graouts> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | graouts, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | |||
Bug Blocks: | 222295, 250970 |
Antoine Quint
For threaded animation resolution, we introduced an `AcceleratedTimeline` class that did the work of updating accelerated effect stacks for targeted elements that were eligible to run their animation effects in the UI process. To support scroll-driven animations using threaded animation resolution, we will need to send animation timelines over to the UI process and will want to use the term `AcceleratedTimeline` for this purpose. So we should rename `AcceleratedTimeline` to something more appropriate. Additionally, while refactoring, we should move it off `Document` and onto `AnimationTimelinesController` which is already held by `Document`.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Antoine Quint
Pull request: https://github.com/WebKit/WebKit/pull/36320
Radar WebKit Bug Importer
<rdar://problem/139427259>
EWS
Committed 286278@main (6f4b5d58b444): <https://commits.webkit.org/286278@main>
Reviewed commits have been landed. Closing PR #36320 and removing active labels.