Remove react_native_new_architecture.enable_runtimescheduler_in_turbomodule_android flag
Summary: changelog: [internal] Turbomodules with RuntimeScheduler on Android are shipped. Let's remove the flag. jest_e2e[run_all_tests] Reviewed By: RSNara Differential Revision: D38456492 fbshipit-source-id: 3dd77c4dc644f1f84b5b74f346fefd701d4cb515
This commit is contained in:
Родитель
051a67784f
Коммит
4899f8c5e6
|
@ -108,8 +108,7 @@ public class CatalystInstanceImpl implements CatalystInstance {
|
|||
// C++ parts
|
||||
private final HybridData mHybridData;
|
||||
|
||||
private static native HybridData initHybrid(
|
||||
boolean enableRuntimeScheduler, boolean enableRuntimeSchedulerInTurboModule);
|
||||
private static native HybridData initHybrid(boolean enableRuntimeScheduler);
|
||||
|
||||
public native CallInvokerHolderImpl getJSCallInvokerHolder();
|
||||
|
||||
|
@ -124,15 +123,7 @@ public class CatalystInstanceImpl implements CatalystInstance {
|
|||
FLog.d(ReactConstants.TAG, "Initializing React Xplat Bridge.");
|
||||
Systrace.beginSection(TRACE_TAG_REACT_JAVA_BRIDGE, "createCatalystInstanceImpl");
|
||||
|
||||
if (ReactFeatureFlags.enableRuntimeSchedulerInTurboModule
|
||||
&& !ReactFeatureFlags.enableRuntimeScheduler) {
|
||||
Assertions.assertUnreachable();
|
||||
}
|
||||
|
||||
mHybridData =
|
||||
initHybrid(
|
||||
ReactFeatureFlags.enableRuntimeScheduler,
|
||||
ReactFeatureFlags.enableRuntimeSchedulerInTurboModule);
|
||||
mHybridData = initHybrid(ReactFeatureFlags.enableRuntimeScheduler);
|
||||
|
||||
mReactQueueConfiguration =
|
||||
ReactQueueConfigurationImpl.create(
|
||||
|
|
|
@ -72,8 +72,6 @@ public class ReactFeatureFlags {
|
|||
|
||||
public static boolean enableRuntimeScheduler = false;
|
||||
|
||||
public static boolean enableRuntimeSchedulerInTurboModule = false;
|
||||
|
||||
/** Feature flag to configure eager attachment of the root view/initialisation of the JS code */
|
||||
public static boolean enableEagerRootViewAttachment = false;
|
||||
|
||||
|
|
|
@ -95,19 +95,13 @@ class JInstanceCallback : public InstanceCallback {
|
|||
jni::local_ref<CatalystInstanceImpl::jhybriddata>
|
||||
CatalystInstanceImpl::initHybrid(
|
||||
jni::alias_ref<jclass>,
|
||||
bool enableRuntimeScheduler,
|
||||
bool enableRuntimeSchedulerInTurboModule) {
|
||||
return makeCxxInstance(
|
||||
enableRuntimeScheduler, enableRuntimeSchedulerInTurboModule);
|
||||
bool enableRuntimeScheduler) {
|
||||
return makeCxxInstance(enableRuntimeScheduler);
|
||||
}
|
||||
|
||||
CatalystInstanceImpl::CatalystInstanceImpl(
|
||||
bool enableRuntimeScheduler,
|
||||
bool enableRuntimeSchedulerInTurboModule)
|
||||
CatalystInstanceImpl::CatalystInstanceImpl(bool enableRuntimeScheduler)
|
||||
: instance_(std::make_unique<Instance>()),
|
||||
enableRuntimeScheduler_(enableRuntimeScheduler),
|
||||
enableRuntimeSchedulerInTurboModule_(
|
||||
enableRuntimeScheduler && enableRuntimeSchedulerInTurboModule) {}
|
||||
enableRuntimeScheduler_(enableRuntimeScheduler) {}
|
||||
|
||||
void CatalystInstanceImpl::warnOnLegacyNativeModuleSystemUse() {
|
||||
CxxNativeModule::setShouldWarnOnUse(true);
|
||||
|
@ -382,7 +376,7 @@ void CatalystInstanceImpl::handleMemoryPressure(int pressureLevel) {
|
|||
jni::alias_ref<CallInvokerHolder::javaobject>
|
||||
CatalystInstanceImpl::getJSCallInvokerHolder() {
|
||||
if (!jsCallInvokerHolder_) {
|
||||
if (enableRuntimeSchedulerInTurboModule_) {
|
||||
if (enableRuntimeScheduler_) {
|
||||
auto runtimeScheduler = getRuntimeScheduler();
|
||||
auto runtimeSchedulerCallInvoker =
|
||||
std::make_shared<RuntimeSchedulerCallInvoker>(
|
||||
|
|
|
@ -39,8 +39,7 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
|
|||
|
||||
static jni::local_ref<jhybriddata> initHybrid(
|
||||
jni::alias_ref<jclass>,
|
||||
bool enableRuntimeScheduler,
|
||||
bool enableRuntimeSchedulerInTurboModule);
|
||||
bool enableRuntimeScheduler);
|
||||
|
||||
static void registerNatives();
|
||||
|
||||
|
@ -51,9 +50,7 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
|
|||
private:
|
||||
friend HybridBase;
|
||||
|
||||
CatalystInstanceImpl(
|
||||
bool enableRuntimeScheduler,
|
||||
bool enableRuntimeSchedulerInTurboModule);
|
||||
CatalystInstanceImpl(bool enableRuntimeScheduler);
|
||||
|
||||
void initializeBridge(
|
||||
jni::alias_ref<ReactCallback::javaobject> callback,
|
||||
|
@ -122,7 +119,6 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
|
|||
jni::global_ref<JRuntimeScheduler::javaobject> runtimeScheduler_;
|
||||
|
||||
bool const enableRuntimeScheduler_;
|
||||
bool const enableRuntimeSchedulerInTurboModule_;
|
||||
};
|
||||
|
||||
} // namespace react
|
||||
|
|
Загрузка…
Ссылка в новой задаче