-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'main' of github.com:Expensify/App into perf/onyx-upgrad…
…e-use-cache-directly
- Loading branch information
Showing
42 changed files
with
1,045 additions
and
814 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
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
25 changes: 25 additions & 0 deletions
25
src/components/AttachmentCarousel/attachmentCarouselPropTypes.js
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,25 @@ | ||
| import PropTypes from 'prop-types'; | ||
| import reportActionPropTypes from '../../pages/home/report/reportActionPropTypes'; | ||
| import reportPropTypes from '../../pages/reportPropTypes'; | ||
|
|
||
| const propTypes = { | ||
| /** source is used to determine the starting index in the array of attachments */ | ||
| source: PropTypes.string, | ||
|
|
||
| /** Callback to update the parent modal's state with a source and name from the attachments array */ | ||
| onNavigate: PropTypes.func, | ||
|
|
||
| /** Object of report actions for this report */ | ||
| reportActions: PropTypes.shape(reportActionPropTypes), | ||
|
|
||
| /** The report currently being looked at */ | ||
| report: reportPropTypes.isRequired, | ||
| }; | ||
|
|
||
| const defaultProps = { | ||
| source: '', | ||
| reportActions: {}, | ||
| onNavigate: () => {}, | ||
| }; | ||
|
|
||
| export {propTypes, defaultProps}; |
Oops, something went wrong.