[OT-PW] Animation integration with new CC impl #17938
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This CL is for Off-thread PaintWorklet animation
integration described in this design doc:
https://docs.google.com/document/d/1a7gO6cBxsJhn53akuJuieUiXvB74vvEmFDyKww8NKdw/edit#
Right now this CL handles composited custom property animations
only, handling native properties should be simpler. Also,
right now we only support custom properties that represent
numerical values.
Here is a summary of the main changes:
A new class AnimatedPaintWorkletTracker is created and LTHI
keeps an instance of it. The new class maintains a hashmap
such that when CC animation ticks, LTHI is able to update the
custom property values for corresponding PaintWorkletInput. And
also that when LTHI needs to dispatch paint job to the worklet
thread, it is able to identify which paint worklets are dirty.
Other changes are mostly plumbing needs associated with the
above changes.
Bug: 956650
Change-Id: If27d6d21cf8fab31e0a2719d81b61cae3d4f3268
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1614120
Commit-Queue: Xida Chen <xidachen@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org>
Cr-Commit-Position: refs/heads/master@{#687049}