diff --git a/Examples/UIExplorer/UIExplorerUnitTests/RCTComponentPropsTests.m b/Examples/UIExplorer/UIExplorerUnitTests/RCTComponentPropsTests.m index 17ed2be2f..b6ab4b769 100644 --- a/Examples/UIExplorer/UIExplorerUnitTests/RCTComponentPropsTests.m +++ b/Examples/UIExplorer/UIExplorerUnitTests/RCTComponentPropsTests.m @@ -99,6 +99,7 @@ RCT_CUSTOM_VIEW_PROPERTY(customProp, NSString, RCTPropsTestView) _bridge = [[RCTBridge alloc] initWithBundleURL:[bundle URLForResource:@"UIExplorerUnitTestsBundle" withExtension:@"js"] moduleProvider:nil launchOptions:nil]; + RUN_RUNLOOP_WHILE(_bridge.isLoading); } - (void)testSetProps diff --git a/React/Base/RCTRootContentView.h b/React/Base/RCTRootContentView.h index f405a8550..f667085e7 100644 --- a/React/Base/RCTRootContentView.h +++ b/React/Base/RCTRootContentView.h @@ -18,11 +18,13 @@ @interface RCTRootContentView : RCTView -@property (nonatomic, readonly) BOOL contentHasAppeared; +@property (nonatomic, readonly, weak) RCTBridge *bridge; +@property (nonatomic, readonly, assign) BOOL contentHasAppeared; @property (nonatomic, readonly, strong) RCTTouchHandler *touchHandler; +@property (nonatomic, readonly, assign) CGSize availableSize; + @property (nonatomic, assign) BOOL passThroughTouches; @property (nonatomic, assign) RCTRootViewSizeFlexibility sizeFlexibility; -@property (nonatomic, readonly) CGSize availableSize; - (instancetype)initWithFrame:(CGRect)frame bridge:(RCTBridge *)bridge diff --git a/React/Base/RCTRootContentView.m b/React/Base/RCTRootContentView.m index f4fb62b53..57ebdd201 100644 --- a/React/Base/RCTRootContentView.m +++ b/React/Base/RCTRootContentView.m @@ -19,7 +19,6 @@ @implementation RCTRootContentView { - __weak RCTBridge *_bridge; UIColor *_backgroundColor; } diff --git a/React/Base/RCTRootView.m b/React/Base/RCTRootView.m index 9870ebfae..e077c5141 100644 --- a/React/Base/RCTRootView.m +++ b/React/Base/RCTRootView.m @@ -92,11 +92,10 @@ NSString *const RCTContentDidAppearNotification = @"RCTContentDidAppearNotificat } #endif - if (!_bridge.loading) { - [self bundleFinishedLoading:([_bridge batchedBridge] ?: _bridge)]; - } - [self showLoadingView]; + + // Immediately schedule the application to be started + [self bundleFinishedLoading:[_bridge batchedBridge]]; } RCT_PROFILE_END_EVENT(RCTProfileTagAlways, @""); @@ -259,11 +258,14 @@ RCT_NOT_IMPLEMENTED(- (instancetype)initWithCoder:(NSCoder *)aDecoder) // Use the (batched) bridge that's sent in the notification payload, so the // RCTRootContentView is scoped to the right bridge RCTBridge *bridge = notification.userInfo[@"bridge"]; - [self bundleFinishedLoading:bridge]; + if (bridge != _contentView.bridge) { + [self bundleFinishedLoading:bridge]; + } } - (void)bundleFinishedLoading:(RCTBridge *)bridge { + RCTAssert(bridge != nil, @"Bridge cannot be nil"); if (!bridge.valid) { return; } diff --git a/React/Modules/RCTUIManager.m b/React/Modules/RCTUIManager.m index ea7e59803..38d093022 100644 --- a/React/Modules/RCTUIManager.m +++ b/React/Modules/RCTUIManager.m @@ -1207,7 +1207,7 @@ RCT_EXPORT_METHOD(dispatchViewManagerCommand:(nonnull NSNumber *)reactTag { // If there is an active batch layout will happen when batch finished, so we will wait for that. // Otherwise we immidiately trigger layout. - if (![_bridge isBatchActive]) { + if (![_bridge isBatchActive] && ![_bridge isLoading]) { [self _layoutAndMount]; } }