-
Notifications
You must be signed in to change notification settings - Fork 3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #18073 from margelo/@terrysahaidak/upgrade-reanimated
Upgrade reanimated to 3.1.0
- Loading branch information
Showing
9 changed files
with
232 additions
and
41 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,177 @@ | ||
| diff --git a/node_modules/react-native-reanimated/ios/REANodesManager.mm b/node_modules/react-native-reanimated/ios/REANodesManager.mm | ||
| index 26bb253..4108293 100644 | ||
| --- a/node_modules/react-native-reanimated/ios/REANodesManager.mm | ||
| +++ b/node_modules/react-native-reanimated/ios/REANodesManager.mm | ||
| @@ -85,19 +85,77 @@ - (void)runSyncUIUpdatesWithObserver:(id<RCTUIManagerObserver>)observer | ||
|
|
||
| @end | ||
|
|
||
| -@interface REANodesManager () <RCTUIManagerObserver> | ||
| +#ifndef RCT_NEW_ARCH_ENABLED | ||
|
|
||
| +@interface REASyncUpdateObserver : NSObject <RCTUIManagerObserver> | ||
| @end | ||
|
|
||
| +@implementation REASyncUpdateObserver { | ||
| + volatile void (^_mounting)(void); | ||
| + volatile BOOL _waitTimedOut; | ||
| + dispatch_semaphore_t _semaphore; | ||
| +} | ||
| + | ||
| +- (instancetype)init | ||
| +{ | ||
| + self = [super init]; | ||
| + if (self) { | ||
| + _mounting = nil; | ||
| + _waitTimedOut = NO; | ||
| + _semaphore = dispatch_semaphore_create(0); | ||
| + } | ||
| + return self; | ||
| +} | ||
| + | ||
| +- (void)dealloc | ||
| +{ | ||
| + RCTAssert(_mounting == nil, @"Mouting block was set but never executed. This may lead to UI inconsistencies"); | ||
| +} | ||
| + | ||
| +- (void)unblockUIThread | ||
| +{ | ||
| + RCTAssertUIManagerQueue(); | ||
| + dispatch_semaphore_signal(_semaphore); | ||
| +} | ||
| + | ||
| +- (void)waitAndMountWithTimeout:(NSTimeInterval)timeout | ||
| +{ | ||
| + RCTAssertMainQueue(); | ||
| + long result = dispatch_semaphore_wait(_semaphore, dispatch_time(DISPATCH_TIME_NOW, timeout * NSEC_PER_SEC)); | ||
| + if (result != 0) { | ||
| + @synchronized(self) { | ||
| + _waitTimedOut = YES; | ||
| + } | ||
| + } | ||
| + if (_mounting) { | ||
| + _mounting(); | ||
| + _mounting = nil; | ||
| + } | ||
| +} | ||
| + | ||
| +- (BOOL)uiManager:(RCTUIManager *)manager performMountingWithBlock:(RCTUIManagerMountingBlock)block | ||
| +{ | ||
| + RCTAssertUIManagerQueue(); | ||
| + @synchronized(self) { | ||
| + if (_waitTimedOut) { | ||
| + return NO; | ||
| + } else { | ||
| + _mounting = block; | ||
| + return YES; | ||
| + } | ||
| + } | ||
| +} | ||
| + | ||
| +@end | ||
| + | ||
| +#endif | ||
| + | ||
| @implementation REANodesManager { | ||
| CADisplayLink *_displayLink; | ||
| BOOL _wantRunUpdates; | ||
| NSMutableArray<REAOnAnimationCallback> *_onAnimationCallbacks; | ||
| BOOL _tryRunBatchUpdatesSynchronously; | ||
| REAEventHandler _eventHandler; | ||
| - volatile void (^_mounting)(void); | ||
| - NSObject *_syncLayoutUpdatesWaitLock; | ||
| - volatile BOOL _syncLayoutUpdatesWaitTimedOut; | ||
| NSMutableDictionary<NSNumber *, ComponentUpdate *> *_componentUpdateBuffer; | ||
| NSMutableDictionary<NSNumber *, UIView *> *_viewRegistry; | ||
| #ifdef RCT_NEW_ARCH_ENABLED | ||
| @@ -125,7 +183,6 @@ - (nonnull instancetype)initWithModule:(REAModule *)reanimatedModule | ||
| _operationsInBatch = [NSMutableDictionary new]; | ||
| _componentUpdateBuffer = [NSMutableDictionary new]; | ||
| _viewRegistry = [_uiManager valueForKey:@"_viewRegistry"]; | ||
| - _syncLayoutUpdatesWaitLock = [NSObject new]; | ||
| } | ||
|
|
||
| _displayLink = [CADisplayLink displayLinkWithTarget:self selector:@selector(onAnimationFrame:)]; | ||
| @@ -241,19 +298,6 @@ - (void)onAnimationFrame:(CADisplayLink *)displayLink | ||
| } | ||
| } | ||
|
|
||
| -- (BOOL)uiManager:(RCTUIManager *)manager performMountingWithBlock:(RCTUIManagerMountingBlock)block | ||
| -{ | ||
| - RCTAssert(_mounting == nil, @"Mouting block is expected to not be set"); | ||
| - @synchronized(_syncLayoutUpdatesWaitLock) { | ||
| - if (_syncLayoutUpdatesWaitTimedOut) { | ||
| - return NO; | ||
| - } else { | ||
| - _mounting = block; | ||
| - return YES; | ||
| - } | ||
| - } | ||
| -} | ||
| - | ||
| - (void)performOperations | ||
| { | ||
| #ifdef RCT_NEW_ARCH_ENABLED | ||
| @@ -268,8 +312,7 @@ - (void)performOperations | ||
| _tryRunBatchUpdatesSynchronously = NO; | ||
|
|
||
| __weak __typeof__(self) weakSelf = self; | ||
| - dispatch_semaphore_t semaphore = dispatch_semaphore_create(0); | ||
| - _syncLayoutUpdatesWaitTimedOut = NO; | ||
| + REASyncUpdateObserver *syncUpdateObserver = [REASyncUpdateObserver new]; | ||
| RCTExecuteOnUIManagerQueue(^{ | ||
| __typeof__(self) strongSelf = weakSelf; | ||
| if (strongSelf == nil) { | ||
| @@ -278,7 +321,7 @@ - (void)performOperations | ||
| BOOL canUpdateSynchronously = trySynchronously && ![strongSelf.uiManager hasEnqueuedUICommands]; | ||
|
|
||
| if (!canUpdateSynchronously) { | ||
| - dispatch_semaphore_signal(semaphore); | ||
| + [syncUpdateObserver unblockUIThread]; | ||
| } | ||
|
|
||
| for (int i = 0; i < copiedOperationsQueue.count; i++) { | ||
| @@ -286,8 +329,8 @@ - (void)performOperations | ||
| } | ||
|
|
||
| if (canUpdateSynchronously) { | ||
| - [strongSelf.uiManager runSyncUIUpdatesWithObserver:strongSelf]; | ||
| - dispatch_semaphore_signal(semaphore); | ||
| + [strongSelf.uiManager runSyncUIUpdatesWithObserver:syncUpdateObserver]; | ||
| + [syncUpdateObserver unblockUIThread]; | ||
| } | ||
| // In case canUpdateSynchronously=true we still have to send uiManagerWillPerformMounting event | ||
| // to observers because some components (e.g. TextInput) update their UIViews only on that event. | ||
| @@ -298,17 +341,7 @@ - (void)performOperations | ||
| // from CADisplayLink but it is easier to hardcode it for the time being. | ||
| // The reason why we use frame duration here is that if takes longer than one frame to complete layout tasks | ||
| // there is no point of synchronizing layout with the UI interaction as we get that one frame delay anyways. | ||
| - long result = dispatch_semaphore_wait(semaphore, dispatch_time(DISPATCH_TIME_NOW, 16 * NSEC_PER_MSEC)); | ||
| - if (result != 0) { | ||
| - @synchronized(_syncLayoutUpdatesWaitLock) { | ||
| - _syncLayoutUpdatesWaitTimedOut = YES; | ||
| - } | ||
| - } | ||
| - } | ||
| - | ||
| - if (_mounting) { | ||
| - _mounting(); | ||
| - _mounting = nil; | ||
| + [syncUpdateObserver waitAndMountWithTimeout:0.016]; | ||
| } | ||
| } | ||
| _wantRunUpdates = NO; | ||
| diff --git a/node_modules/react-native-reanimated/mock.js b/node_modules/react-native-reanimated/mock.js | ||
| index 68b20d2..b088001 100644 | ||
| --- a/node_modules/react-native-reanimated/mock.js | ||
| +++ b/node_modules/react-native-reanimated/mock.js | ||
| @@ -41,6 +41,9 @@ const Reanimated = { | ||
| createAnimatedComponent: (Component) => Component, | ||
| addWhitelistedUIProps: NOOP, | ||
| addWhitelistedNativeProps: NOOP, | ||
| + | ||
| + // used by react-navigation fork | ||
| + isConfigured: () => true, | ||
| }; | ||
|
|
||
| module.exports = { |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.