From 4cba3abf06d133607aa3b67d743c2d48f066c965 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Gregorczyk?= Date: Fri, 1 Sep 2017 09:01:17 -0700 Subject: [PATCH] Fix NPE in ReactInstaceManager.setupReactContext Reviewed By: AaaChiuuu, mhorowitz Differential Revision: D5724464 fbshipit-source-id: 80d0b97f4294f6894e0d034dda6dd1a3db4b1341 --- .../facebook/react/ReactInstanceManager.java | 39 ++++++++++--------- 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java b/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java index 52e44d387..29c9394d5 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java @@ -903,25 +903,28 @@ public class ReactInstanceManager { }); Systrace.endSection(TRACE_TAG_REACT_JAVA_BRIDGE); ReactMarker.logMarker(SETUP_REACT_CONTEXT_END); - mCurrentReactContext.runOnJSQueueThread(new Runnable() { - @Override - public void run() { - Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); - } - }); - mCurrentReactContext.runOnNativeModulesQueueThread(new Runnable() { - @Override - public void run() { - Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); - } - }); + reactContext.runOnJSQueueThread( + new Runnable() { + @Override + public void run() { + Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); + } + }); + reactContext.runOnNativeModulesQueueThread( + new Runnable() { + @Override + public void run() { + Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); + } + }); if (mUseSeparateUIBackgroundThread) { - mCurrentReactContext.runOnUiBackgroundQueueThread(new Runnable() { - @Override - public void run() { - Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); - } - }); + reactContext.runOnUiBackgroundQueueThread( + new Runnable() { + @Override + public void run() { + Process.setThreadPriority(Process.THREAD_PRIORITY_DEFAULT); + } + }); } }