status-im/react-native
2
0
Fork 0
mirror of https://github.com/status-im/react-native.git synced 2025-02-26 16:10:58 +00:00
Code Issues Projects Releases Wiki Activity

Prevent deadlock after bridge reload

Browse Source 
Reviewed By: dcaspi

Differential Revision: D4551097

fbshipit-source-id: 33b43b2691d4e595d3c09c9a5b101ce458c4d6d7
This commit is contained in:
Pieter De Baets 2017-02-15 06:12:44 -08:00 committed by Facebook Github Bot
parent a2000d1200
commit bbe0c45c19
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
React/CxxBridge/RCTCxxBridge.mm
View File

@ -25,9 +25,6 @@
#import <React/RCTDevLoadingView.h>
#import <React/RCTDevMenu.h>
#import <React/RCTDisplayLink.h>
#ifdef WITH_FBSYSTRACE
#import <React/RCTFBSystrace.h>
#endif
#import <React/RCTJavaScriptLoader.h>
#import <React/RCTLog.h>
#import <React/RCTModuleData.h>
@ -47,6 +44,10 @@
#import "RCTNativeModule.h"
#import "RCTObjcExecutor.h"
#ifdef WITH_FBSYSTRACE
#import <React/RCTFBSystrace.h>
#endif
#define RCTAssertJSThread() \
RCTAssert(self.executorClass || self->_jsThread == [NSThread currentThread], \
@"This method must be called on JS thread")
@ -578,16 +579,15 @@ static NSError *tryAndReturnError(dispatch_block_t block) {
for (RCTModuleData *moduleData in _moduleDataByID) {
// TODO mhorowitz #10487027: unwrap C++ modules and register them directly.
if ([moduleData.moduleClass isSubclassOfClass:[RCTCxxModule class]]) {
RCTCxxModule *cxxInstance = moduleData.instance;
// If a module does not support automatic instantiation, and
// wasn't provided as an extra module, it may not have an
// instance. If so, skip it.
if (!cxxInstance) {
if (![moduleData hasInstance]) {
continue;
}
modules.emplace_back(
new QueueNativeModule(self, std::make_unique<CxxNativeModule>(
_reactInstance, [cxxInstance move])));
_reactInstance, [(RCTCxxModule *)(moduleData.instance) move])));
} else {
modules.emplace_back(new RCTNativeModule(self, moduleData));
}