Skip to content

Split families by surfaceId in Animation Backend#54424

Closed
bartlomiejbloniarz wants to merge 3 commits into
mainfrom
export-D84055753
Closed

Split families by surfaceId in Animation Backend#54424
bartlomiejbloniarz wants to merge 3 commits into
mainfrom
export-D84055753

Conversation

@bartlomiejbloniarz

Copy link
Copy Markdown
Contributor

Summary: AnimationBackend might be handling mulitple surfaces at once, so we need to separate the updates, by SurfaceId

Reviewed By: zeyap

Differential Revision: D84055753

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 6, 2025
@facebook-github-bot facebook-github-bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Nov 6, 2025
@meta-codesync

meta-codesync Bot commented Nov 6, 2025

Copy link
Copy Markdown

@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating Diff in D84055753.

@github-actions

github-actions Bot commented Nov 6, 2025

Copy link
Copy Markdown

Job Summary for Gradle

Test All :: run_fantom_tests
Gradle Root Project Requested Tasks Gradle Version Build Outcome Build Scan®
react-native-github :private:react-native-fantom:buildFantomTester 9.0.0 Build Scan not published

@facebook-github-bot facebook-github-bot force-pushed the export-D84055753 branch 2 times, most recently from a7304ea to 6ef45ad Compare November 6, 2025 10:29
Bartlomiej Bloniarz added 3 commits November 6, 2025 02:51
Summary:

Withouth the backend, Animated triggers the start function once per frame (by updating an atomic bool). We now do the same with animation backend.

Reviewed By: zeyap

Differential Revision: D85234259
Summary:

Animation frames on Android are tiggered from UIManagerNativeAnimatedDelegate::runAnimationFrame, so we implement that for the backend

Reviewed By: zeyap

Differential Revision: D84055754
Summary:

AnimationBackend might be handling mulitple surfaces at once, so we need to separate the updates, by `SurfaceId`

Reviewed By: zeyap

Differential Revision: D84055753
@meta-codesync meta-codesync Bot closed this in 5c078bd Nov 6, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 6, 2025
@meta-codesync

meta-codesync Bot commented Nov 6, 2025

Copy link
Copy Markdown

This pull request has been merged in 5c078bd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants