From a31a2910a15cb48ac6409b1f8a908412d1587ab6 Mon Sep 17 00:00:00 2001 From: Alexey Lang Date: Wed, 7 Sep 2016 13:01:47 -0700 Subject: [PATCH] Make bridge logger thread-safe and fix noisy in bridge core time Reviewed By: javache Differential Revision: D3828301 fbshipit-source-id: 29efdef2fb2c601984c27c4d0a69e1542bbff385 --- React/Base/RCTBatchedBridge.m | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/React/Base/RCTBatchedBridge.m b/React/Base/RCTBatchedBridge.m index 1128b3590acdd5..867626233abc17 100644 --- a/React/Base/RCTBatchedBridge.m +++ b/React/Base/RCTBatchedBridge.m @@ -544,6 +544,8 @@ - (void)executeSourceCode:(NSData *)sourceCode NSRunLoop *targetRunLoop = [self->_javaScriptExecutor isKindOfClass:[RCTJSCExecutor class]] ? [NSRunLoop currentRunLoop] : [NSRunLoop mainRunLoop]; [self->_displayLink addToRunLoop:targetRunLoop]; + [self->_performanceLogger markStopForTag:RCTPLBridgeStartup]; + // Perform the notification on the main thread, so we can't run into // timing issues with RCTRootView dispatch_async(dispatch_get_main_queue(), ^{ @@ -571,7 +573,6 @@ - (void)executeSourceCode:(NSData *)sourceCode - (void)_flushPendingCalls { RCTAssertJSThread(); - [_performanceLogger markStopForTag:RCTPLBridgeStartup]; RCT_PROFILE_BEGIN_EVENT(0, @"Processing pendingCalls", @{ @"count": @(_pendingCalls.count) }); _loading = NO;