Skip to content

Commit b19bf2b

Browse files
philIipfacebook-github-bot
authored andcommitted
ship shared module queue (facebook#44916)
Summary: Pull Request resolved: facebook#44916 Changelog: [Internal] reached ship decision last year, getting rid of the gating here Reviewed By: javache Differential Revision: D58476997 fbshipit-source-id: 0ad52d29ac79b1751d6f10e67dc6886bf88e5ba4
1 parent f7aea0c commit b19bf2b

3 files changed

Lines changed: 2 additions & 28 deletions

File tree

packages/react-native/React/Base/RCTBridge.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,6 @@ void RCTEnableFabricInteropLayer(BOOL enabled);
6161
BOOL RCTTurboModuleSyncVoidMethodsEnabled(void);
6262
void RCTEnableTurboModuleSyncVoidMethods(BOOL enabled);
6363

64-
// Use a shared queue for executing module methods
65-
BOOL RCTTurboModuleSharedQueueEnabled(void);
66-
void RCTEnableTurboModuleSharedQueue(BOOL enabled);
67-
6864
BOOL RCTUIManagerDispatchAccessibilityManagerInitOntoMain(void);
6965
void RCTUIManagerSetDispatchAccessibilityManagerInitOntoMain(BOOL enabled);
7066

packages/react-native/React/Base/RCTBridge.mm

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -167,18 +167,6 @@ void RCTEnableTurboModuleSyncVoidMethods(BOOL enabled)
167167
gTurboModuleEnableSyncVoidMethods = enabled;
168168
}
169169

170-
// Use a shared queue for executing module methods
171-
static BOOL gTurboModuleEnableSharedQueue = NO;
172-
BOOL RCTTurboModuleSharedQueueEnabled(void)
173-
{
174-
return gTurboModuleEnableSharedQueue;
175-
}
176-
177-
void RCTEnableTurboModuleSharedQueue(BOOL enabled)
178-
{
179-
gTurboModuleEnableSharedQueue = enabled;
180-
}
181-
182170
BOOL kDispatchAccessibilityManagerInitOntoMain = NO;
183171
BOOL RCTUIManagerDispatchAccessibilityManagerInitOntoMain(void)
184172
{

packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,6 @@ @implementation RCTTurboModuleManager {
217217
RCTBridgeProxy *_bridgeProxy;
218218
RCTBridgeModuleDecorator *_bridgeModuleDecorator;
219219

220-
BOOL _enableSharedModuleQueue;
221220
dispatch_queue_t _sharedModuleQueue;
222221
}
223222

@@ -234,11 +233,7 @@ - (instancetype)initWithBridge:(RCTBridge *)bridge
234233
_bridgeProxy = bridgeProxy;
235234
_bridgeModuleDecorator = bridgeModuleDecorator;
236235
_invalidating = false;
237-
_enableSharedModuleQueue = RCTTurboModuleSharedQueueEnabled();
238-
239-
if (_enableSharedModuleQueue) {
240-
_sharedModuleQueue = dispatch_queue_create("com.meta.react.turbomodulemanager.queue", DISPATCH_QUEUE_SERIAL);
241-
}
236+
_sharedModuleQueue = dispatch_queue_create("com.meta.react.turbomodulemanager.queue", DISPATCH_QUEUE_SERIAL);
242237

243238
if (RCTTurboModuleInteropEnabled()) {
244239
// TODO(T174674274): Implement lazy loading of legacy modules in the new architecture.
@@ -728,12 +723,7 @@ - (BOOL)_shouldCreateObjCModule:(Class)moduleClass
728723
* following if condition's block.
729724
*/
730725
if (!methodQueue) {
731-
if (_enableSharedModuleQueue) {
732-
methodQueue = _sharedModuleQueue;
733-
} else {
734-
NSString *methodQueueName = [NSString stringWithFormat:@"com.facebook.react.%sQueue", moduleName];
735-
methodQueue = dispatch_queue_create(methodQueueName.UTF8String, DISPATCH_QUEUE_SERIAL);
736-
}
726+
methodQueue = _sharedModuleQueue;
737727

738728
if (moduleHasMethodQueueGetter) {
739729
/**

0 commit comments

Comments
 (0)