From 0357d0de64133f9ce2332f05fbf943a460b5e0f1 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Sun, 25 Nov 2018 17:18:09 -0800 Subject: [PATCH] Ensure thread safety in the exeuction of RuntimeExecutor Summary: This diff ensures thread safety for operations invoked by the runtime executor Reviewed By: shergin Differential Revision: D13136340 fbshipit-source-id: 119092dff29b37f39d4bcdcc34f1c34d638b7e07 --- .../main/java/com/facebook/react/fabric/FabricBinding.java | 6 +++++- ReactAndroid/src/main/jni/react/jni/BUCK | 1 + 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricBinding.java b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricBinding.java index 29ed2c9c39c..22e71c275a9 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricBinding.java +++ b/ReactAndroid/src/main/java/com/facebook/react/fabric/FabricBinding.java @@ -7,12 +7,16 @@ package com.facebook.react.fabric; import com.facebook.react.bridge.JavaScriptContextHolder; +import com.facebook.react.bridge.queue.MessageQueueThread; public interface FabricBinding { // TODO: T31905686 change types of UIManager and EventBeatManager when moving to OSS void register( - JavaScriptContextHolder jsContext, FabricBinder fabricBinder, Object eventBeatManager); + JavaScriptContextHolder jsContext, + FabricBinder fabricBinder, + Object eventBeatManager, + MessageQueueThread jsMessageQueueThread); void unregister(); } diff --git a/ReactAndroid/src/main/jni/react/jni/BUCK b/ReactAndroid/src/main/jni/react/jni/BUCK index b2776f17043..141f76ea296 100644 --- a/ReactAndroid/src/main/jni/react/jni/BUCK +++ b/ReactAndroid/src/main/jni/react/jni/BUCK @@ -6,6 +6,7 @@ EXPORTED_HEADERS = [ "CxxSharedModuleWrapper.h", "JavaModuleWrapper.h", "JavaScriptExecutorHolder.h", + "JMessageQueueThread.h", "JReactMarker.h", "JSLoader.h", "JSLogging.h",