Commit 55248b3
Revert RCTAnimatedModuleProvider change from D94244698 (facebook#56100)
Summary:
Pull Request resolved: facebook#56100
## Changelog:
[iOS] [Fixed] - Revert RCTAnimatedModuleProvider change from D94244698
D94244698 added a guard in RCTAnimatedModuleProvider that returns nullptr
when useSharedAnimatedBackend() is true, expecting DefaultTurboModules to
handle AnimatedModule creation instead.
However, on iOS the TurboModule resolution chain in RCTReactNativeFactory
delegates to the app-provided getTurboModule:jsInvoker: and returns whatever
the delegate returns — even nullptr — without falling through to
DefaultTurboModules.
This causes `Invariant Violation: Native animated module is not available`
on any surface using Animated.View (e.g. Marketplace PDP) when
react_fabric.enable_shared_animated_backend_ios is enabled.
Revert the RCTAnimatedModuleProvider change from D94244698 so it always
provides AnimatedModule when cxxNativeAnimatedEnabled is true, regardless
of useSharedAnimatedBackend. The shared backend path in DefaultTurboModules
still exists as fallback for non-iOS platforms.
Reviewed By: christophpurrer
Differential Revision: D96611917
fbshipit-source-id: e01cf5c80dc4cbf30afac6bcf414616c15bfaddb1 parent af0cccc commit 55248b3
File tree
1 file changed
+1
-4
lines changed- packages/react-native/ReactApple/RCTAnimatedModuleProvider
1 file changed
+1
-4
lines changedLines changed: 1 addition & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
74 | | - | |
75 | | - | |
76 | | - | |
| 73 | + | |
77 | 74 | | |
78 | 75 | | |
79 | 76 | | |
| |||
0 commit comments