Skip to content

Commit 08b14f4

Browse files
committed
Revert "Fix ReactRootView mount/unmount race condition"
This reverts commit 6524591.
1 parent 53672ef commit 08b14f4

File tree

3 files changed

+15
-91
lines changed

3 files changed

+15
-91
lines changed

ReactAndroid/src/androidTest/java/com/facebook/react/tests/core/BUCK

-29
This file was deleted.

ReactAndroid/src/androidTest/java/com/facebook/react/tests/core/ReactInstanceManagerTest.java

-46
This file was deleted.

ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java

+15-16
Original file line numberDiff line numberDiff line change
@@ -732,9 +732,8 @@ public void attachRootView(ReactRootView rootView) {
732732
@ThreadConfined(UI)
733733
public void detachRootView(ReactRootView rootView) {
734734
UiThreadUtil.assertOnUiThread();
735-
if (mAttachedRootViews.contains(rootView)) {
735+
if (mAttachedRootViews.remove(rootView)) {
736736
ReactContext currentContext = getCurrentReactContext();
737-
mAttachedRootViews.remove(rootView);
738737
if (currentContext != null && currentContext.hasActiveCatalystInstance()) {
739738
detachViewFromInstance(rootView, currentContext.getCatalystInstance());
740739
}
@@ -960,22 +959,22 @@ private void setupReactContext(final ReactApplicationContext reactContext) {
960959
Systrace.beginSection(TRACE_TAG_REACT_JAVA_BRIDGE, "setupReactContext");
961960
synchronized (mReactContextLock) {
962961
mCurrentReactContext = Assertions.assertNotNull(reactContext);
963-
CatalystInstance catalystInstance =
964-
Assertions.assertNotNull(reactContext.getCatalystInstance());
965-
966-
catalystInstance.initialize();
967-
mDevSupportManager.onNewReactContextCreated(reactContext);
968-
mMemoryPressureRouter.addMemoryPressureListener(catalystInstance);
969-
moveReactContextToCurrentLifecycleState();
970-
971-
ReactMarker.logMarker(ATTACH_MEASURED_ROOT_VIEWS_START);
972-
synchronized (mAttachedRootViews) {
973-
for (ReactRootView rootView : mAttachedRootViews) {
974-
attachRootViewToInstance(rootView);
975-
}
962+
}
963+
CatalystInstance catalystInstance =
964+
Assertions.assertNotNull(reactContext.getCatalystInstance());
965+
966+
catalystInstance.initialize();
967+
mDevSupportManager.onNewReactContextCreated(reactContext);
968+
mMemoryPressureRouter.addMemoryPressureListener(catalystInstance);
969+
moveReactContextToCurrentLifecycleState();
970+
971+
ReactMarker.logMarker(ATTACH_MEASURED_ROOT_VIEWS_START);
972+
synchronized (mAttachedRootViews) {
973+
for (ReactRootView rootView : mAttachedRootViews) {
974+
attachRootViewToInstance(rootView);
976975
}
977-
ReactMarker.logMarker(ATTACH_MEASURED_ROOT_VIEWS_END);
978976
}
977+
ReactMarker.logMarker(ATTACH_MEASURED_ROOT_VIEWS_END);
979978

980979
ReactInstanceEventListener[] listeners =
981980
new ReactInstanceEventListener[mReactInstanceEventListeners.size()];

0 commit comments

Comments
 (0)