Skip to content

Commit f83e7ea

Browse files
javachefacebook-github-bot
authored andcommitted
Remove enableBufferedCallInvoker feature flag
Summary: Buffered async `CallInvoker` work has been enabled by default and fully rolled out. Make `ReactInstance` always construct `CallInvokerImpl`, remove the scheduler-backed fallback, and delete the generated flag surface. Changelog: [Internal] Differential Revision: D123534701
1 parent d08d8d2 commit f83e7ea

23 files changed

Lines changed: 98 additions & 229 deletions

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlags.kt‎

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<86390dbd6b2c57cb4f8cdd7cb73cc1f7>>
7+
* @generated SignedSource<<da44ac77dae3658d58940f0230a8ead6>>
88
*/
99

1010
/**
@@ -126,12 +126,6 @@ public object ReactNativeFeatureFlags {
126126
@JvmStatic
127127
public fun enableBridgelessArchitecture(): Boolean = accessor.enableBridgelessArchitecture()
128128

129-
/**
130-
* Route async CallInvoker work through the ReactInstance buffered runtime executor, so it is ordered against callable module calls and cannot run before the JS bundle has finished evaluating. invokeSync is unaffected.
131-
*/
132-
@JvmStatic
133-
public fun enableBufferedCallInvoker(): Boolean = accessor.enableBufferedCallInvoker()
134-
135129
/**
136130
* Enable prop iterator setter-style construction of Props in C++ (this flag is not used in Java).
137131
*/

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxAccessor.kt‎

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<3e939eebe846d12cab15905db8efba18>>
7+
* @generated SignedSource<<1f2c6af3ce83f8f56e6fef3eece891bd>>
88
*/
99

1010
/**
@@ -36,7 +36,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
3636
private var enableAndroidAutoOffscreenCompositingForElevationCache: Boolean? = null
3737
private var enableAndroidTextMeasurementOptimizationsCache: Boolean? = null
3838
private var enableBridgelessArchitectureCache: Boolean? = null
39-
private var enableBufferedCallInvokerCache: Boolean? = null
4039
private var enableCppPropsIteratorSetterCache: Boolean? = null
4140
private var enableCustomFocusSearchOnClippedElementsAndroidCache: Boolean? = null
4241
private var enableDestroyShadowTreeRevisionAsyncCache: Boolean? = null
@@ -255,15 +254,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
255254
return cached
256255
}
257256

258-
override fun enableBufferedCallInvoker(): Boolean {
259-
var cached = enableBufferedCallInvokerCache
260-
if (cached == null) {
261-
cached = ReactNativeFeatureFlagsCxxInterop.enableBufferedCallInvoker()
262-
enableBufferedCallInvokerCache = cached
263-
}
264-
return cached
265-
}
266-
267257
override fun enableCppPropsIteratorSetter(): Boolean {
268258
var cached = enableCppPropsIteratorSetterCache
269259
if (cached == null) {

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxInterop.kt‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<da34d40665437b2901de2ca5f0bee7c1>>
7+
* @generated SignedSource<<9fe9b6af3dd1aba706dd87a8dd0a9c57>>
88
*/
99

1010
/**
@@ -60,8 +60,6 @@ public object ReactNativeFeatureFlagsCxxInterop {
6060

6161
@DoNotStrip @JvmStatic public external fun enableBridgelessArchitecture(): Boolean
6262

63-
@DoNotStrip @JvmStatic public external fun enableBufferedCallInvoker(): Boolean
64-
6563
@DoNotStrip @JvmStatic public external fun enableCppPropsIteratorSetter(): Boolean
6664

6765
@DoNotStrip @JvmStatic public external fun enableCustomFocusSearchOnClippedElementsAndroid(): Boolean

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsDefaults.kt‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<0f4c074c11c564f05950eab73a8fd452>>
7+
* @generated SignedSource<<1e15400f6b519fc8e9cd3e576501e7ed>>
88
*/
99

1010
/**
@@ -55,8 +55,6 @@ public open class ReactNativeFeatureFlagsDefaults : ReactNativeFeatureFlagsProvi
5555

5656
override fun enableBridgelessArchitecture(): Boolean = true
5757

58-
override fun enableBufferedCallInvoker(): Boolean = true
59-
6058
override fun enableCppPropsIteratorSetter(): Boolean = false
6159

6260
override fun enableCustomFocusSearchOnClippedElementsAndroid(): Boolean = true

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsLocalAccessor.kt‎

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<013790bd2c3fa8b9ff03e8c5e924bf91>>
7+
* @generated SignedSource<<5f805273ebd792ecec1c284ca074e190>>
88
*/
99

1010
/**
@@ -40,7 +40,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
4040
private var enableAndroidAutoOffscreenCompositingForElevationCache: Boolean? = null
4141
private var enableAndroidTextMeasurementOptimizationsCache: Boolean? = null
4242
private var enableBridgelessArchitectureCache: Boolean? = null
43-
private var enableBufferedCallInvokerCache: Boolean? = null
4443
private var enableCppPropsIteratorSetterCache: Boolean? = null
4544
private var enableCustomFocusSearchOnClippedElementsAndroidCache: Boolean? = null
4645
private var enableDestroyShadowTreeRevisionAsyncCache: Boolean? = null
@@ -275,16 +274,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
275274
return cached
276275
}
277276

278-
override fun enableBufferedCallInvoker(): Boolean {
279-
var cached = enableBufferedCallInvokerCache
280-
if (cached == null) {
281-
cached = currentProvider.enableBufferedCallInvoker()
282-
accessedFeatureFlags.add("enableBufferedCallInvoker")
283-
enableBufferedCallInvokerCache = cached
284-
}
285-
return cached
286-
}
287-
288277
override fun enableCppPropsIteratorSetter(): Boolean {
289278
var cached = enableCppPropsIteratorSetterCache
290279
if (cached == null) {

‎packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsProvider.kt‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<6607c8678df2693c361f6d8091483662>>
7+
* @generated SignedSource<<c2c4ed4f4bca0b82aefc82dfb43617b7>>
88
*/
99

1010
/**
@@ -55,8 +55,6 @@ public interface ReactNativeFeatureFlagsProvider {
5555

5656
@DoNotStrip public fun enableBridgelessArchitecture(): Boolean
5757

58-
@DoNotStrip public fun enableBufferedCallInvoker(): Boolean
59-
6058
@DoNotStrip public fun enableCppPropsIteratorSetter(): Boolean
6159

6260
@DoNotStrip public fun enableCustomFocusSearchOnClippedElementsAndroid(): Boolean

‎packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.cpp‎

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<dad8d48d93f01fc5ec68f70dbd9a9e73>>
7+
* @generated SignedSource<<667c7bb26992905d8a73351b6d5eeadd>>
88
*/
99

1010
/**
@@ -135,12 +135,6 @@ class ReactNativeFeatureFlagsJavaProvider
135135
return method(javaProvider_);
136136
}
137137

138-
bool enableBufferedCallInvoker() override {
139-
static const auto method =
140-
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableBufferedCallInvoker");
141-
return method(javaProvider_);
142-
}
143-
144138
bool enableCppPropsIteratorSetter() override {
145139
static const auto method =
146140
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableCppPropsIteratorSetter");
@@ -663,11 +657,6 @@ bool JReactNativeFeatureFlagsCxxInterop::enableBridgelessArchitecture(
663657
return ReactNativeFeatureFlags::enableBridgelessArchitecture();
664658
}
665659

666-
bool JReactNativeFeatureFlagsCxxInterop::enableBufferedCallInvoker(
667-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
668-
return ReactNativeFeatureFlags::enableBufferedCallInvoker();
669-
}
670-
671660
bool JReactNativeFeatureFlagsCxxInterop::enableCppPropsIteratorSetter(
672661
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
673662
return ReactNativeFeatureFlags::enableCppPropsIteratorSetter();
@@ -1112,9 +1101,6 @@ void JReactNativeFeatureFlagsCxxInterop::registerNatives() {
11121101
makeNativeMethod(
11131102
"enableBridgelessArchitecture",
11141103
JReactNativeFeatureFlagsCxxInterop::enableBridgelessArchitecture),
1115-
makeNativeMethod(
1116-
"enableBufferedCallInvoker",
1117-
JReactNativeFeatureFlagsCxxInterop::enableBufferedCallInvoker),
11181104
makeNativeMethod(
11191105
"enableCppPropsIteratorSetter",
11201106
JReactNativeFeatureFlagsCxxInterop::enableCppPropsIteratorSetter),

‎packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.h‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<7a9a9c1938c3e01f589e05fffe349ad3>>
7+
* @generated SignedSource<<91436689b93d2b7f42006c182f5a1bf7>>
88
*/
99

1010
/**
@@ -78,9 +78,6 @@ class JReactNativeFeatureFlagsCxxInterop
7878
static bool enableBridgelessArchitecture(
7979
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
8080

81-
static bool enableBufferedCallInvoker(
82-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
83-
8481
static bool enableCppPropsIteratorSetter(
8582
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
8683

‎packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.cpp‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<8b23b6ea5f2d377f8126adffd753dae7>>
7+
* @generated SignedSource<<eba4d2684dff3009f3f5b9a106e96ba7>>
88
*/
99

1010
/**
@@ -90,10 +90,6 @@ bool ReactNativeFeatureFlags::enableBridgelessArchitecture() {
9090
return getAccessor().enableBridgelessArchitecture();
9191
}
9292

93-
bool ReactNativeFeatureFlags::enableBufferedCallInvoker() {
94-
return getAccessor().enableBufferedCallInvoker();
95-
}
96-
9793
bool ReactNativeFeatureFlags::enableCppPropsIteratorSetter() {
9894
return getAccessor().enableCppPropsIteratorSetter();
9995
}

‎packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.h‎

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* This source code is licensed under the MIT license found in the
55
* LICENSE file in the root directory of this source tree.
66
*
7-
* @generated SignedSource<<aef7d6ac25fdeb2423ebf4a2a9ac6f2e>>
7+
* @generated SignedSource<<d3b754504fd7d6d0c978faec52d14c5d>>
88
*/
99

1010
/**
@@ -121,11 +121,6 @@ class ReactNativeFeatureFlags {
121121
*/
122122
RN_EXPORT static bool enableBridgelessArchitecture();
123123

124-
/**
125-
* Route async CallInvoker work through the ReactInstance buffered runtime executor, so it is ordered against callable module calls and cannot run before the JS bundle has finished evaluating. invokeSync is unaffected.
126-
*/
127-
RN_EXPORT static bool enableBufferedCallInvoker();
128-
129124
/**
130125
* Enable prop iterator setter-style construction of Props in C++ (this flag is not used in Java).
131126
*/

0 commit comments

Comments
 (0)