mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Revert D44153451: Multisect successfully blamed D44153451 for test or build failures (#36756)
Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/36756 This diff is reverting D44153451 Depends on D44602279 D44153451: [Fabric][Android] Reduce visibility of FabricSoLoader by mdvacca has been identified to be causing the following test or build failures: Tests affected: - [xplat/endtoend/jest-e2e/apps/fb4a/__tests__/consumerwifi/venice/fb4aVeniceAddressSearch-e2e.js](https://www.internalfb.com/intern/test/281475007251570/) Here's the Multisect link: https://www.internalfb.com/multisect/1799601 Here are the tasks that are relevant to this breakage: We're generating a revert to back out the changes in this diff, please note the backout may land if someone accepts it. Reviewed By: mdvacca Differential Revision: D44602284 fbshipit-source-id: 50b3b0f131c6c8c1eec3c23156b33559ffc3931e
This commit is contained in:
committed by
Facebook GitHub Bot
parent
86859e3e38
commit
a2d2696e43
+1
-1
@@ -14,7 +14,7 @@ import com.facebook.react.bridge.ReactMarkerConstants;
|
||||
import com.facebook.soloader.SoLoader;
|
||||
import com.facebook.systrace.Systrace;
|
||||
|
||||
class FabricSoLoader {
|
||||
public class FabricSoLoader {
|
||||
private static volatile boolean sDidInit = false;
|
||||
|
||||
public static void staticInit() {
|
||||
|
||||
+4
-4
@@ -91,10 +91,6 @@ import java.util.concurrent.CopyOnWriteArrayList;
|
||||
@SuppressLint("MissingNativeLoadLibrary")
|
||||
@DoNotStripAny
|
||||
public class FabricUIManager implements UIManager, LifecycleEventListener {
|
||||
static {
|
||||
FabricSoLoader.staticInit();
|
||||
}
|
||||
|
||||
public static final String TAG = FabricUIManager.class.getSimpleName();
|
||||
|
||||
// The IS_DEVELOPMENT_ENVIRONMENT variable is used to log extra data when running fabric in a
|
||||
@@ -155,6 +151,10 @@ public class FabricUIManager implements UIManager, LifecycleEventListener {
|
||||
}
|
||||
};
|
||||
|
||||
static {
|
||||
FabricSoLoader.staticInit();
|
||||
}
|
||||
|
||||
@Nullable private Binding mBinding;
|
||||
@NonNull private final ReactApplicationContext mReactApplicationContext;
|
||||
@NonNull private final MountingManager mMountingManager;
|
||||
|
||||
+5
@@ -12,6 +12,7 @@ import androidx.annotation.NonNull;
|
||||
import com.facebook.jni.HybridData;
|
||||
import com.facebook.proguard.annotations.DoNotStrip;
|
||||
import com.facebook.react.bridge.ReactApplicationContext;
|
||||
import com.facebook.react.fabric.FabricSoLoader;
|
||||
import com.facebook.react.uimanager.events.BatchEventDispatchedListener;
|
||||
|
||||
/**
|
||||
@@ -20,6 +21,10 @@ import com.facebook.react.uimanager.events.BatchEventDispatchedListener;
|
||||
@SuppressLint("MissingNativeLoadLibrary")
|
||||
public class EventBeatManager implements BatchEventDispatchedListener {
|
||||
|
||||
static {
|
||||
FabricSoLoader.staticInit();
|
||||
}
|
||||
|
||||
@DoNotStrip private final HybridData mHybridData;
|
||||
private final ReactApplicationContext mReactApplicationContext;
|
||||
|
||||
|
||||
+5
@@ -14,6 +14,7 @@ import com.facebook.jni.HybridData;
|
||||
import com.facebook.proguard.annotations.DoNotStrip;
|
||||
import com.facebook.react.bridge.NativeMap;
|
||||
import com.facebook.react.bridge.WritableMap;
|
||||
import com.facebook.react.fabric.FabricSoLoader;
|
||||
import com.facebook.react.uimanager.events.EventCategoryDef;
|
||||
|
||||
/**
|
||||
@@ -23,6 +24,10 @@ import com.facebook.react.uimanager.events.EventCategoryDef;
|
||||
@SuppressLint("MissingNativeLoadLibrary")
|
||||
public class EventEmitterWrapper {
|
||||
|
||||
static {
|
||||
FabricSoLoader.staticInit();
|
||||
}
|
||||
|
||||
@DoNotStrip private final HybridData mHybridData;
|
||||
|
||||
private EventEmitterWrapper(HybridData hybridData) {
|
||||
|
||||
Reference in New Issue
Block a user