From 093b745c214c06ded4bbe2f01f7fe3cd55397f13 Mon Sep 17 00:00:00 2001 From: Nick Gerleman Date: Tue, 13 Feb 2024 18:18:20 -0800 Subject: [PATCH] Back out "Migrate flag enablePropIteratorSetter to new system" Summary: Original commit changeset: d6fd1e819abb Original Phabricator Diff: D52810065 Changelog: [Internal] Reviewed By: lunaleaps, mdvacca Differential Revision: D53736651 fbshipit-source-id: 1fcf7aa028101f6439dfe34d879a086e9cae71ff --- .../React/Fabric/RCTSurfacePresenter.mm | 4 + .../ReactAndroid/api/ReactAndroid.api | 1 + .../react/config/ReactFeatureFlags.java | 5 + .../featureflags/ReactNativeFeatureFlags.kt | 8 +- .../ReactNativeFeatureFlagsCxxAccessor.kt | 12 +- .../ReactNativeFeatureFlagsCxxInterop.kt | 4 +- .../ReactNativeFeatureFlagsDefaults.kt | 4 +- .../ReactNativeFeatureFlagsLocalAccessor.kt | 13 +- .../ReactNativeFeatureFlagsProvider.kt | 4 +- .../src/main/jni/react/fabric/Binding.cpp | 2 + .../JReactNativeFeatureFlagsCxxInterop.cpp | 16 +- .../JReactNativeFeatureFlagsCxxInterop.h | 5 +- .../featureflags/ReactNativeFeatureFlags.cpp | 6 +- .../featureflags/ReactNativeFeatureFlags.h | 7 +- .../ReactNativeFeatureFlagsAccessor.cpp | 20 +- .../ReactNativeFeatureFlagsAccessor.h | 6 +- .../ReactNativeFeatureFlagsDefaults.h | 6 +- .../ReactNativeFeatureFlagsProvider.h | 3 +- .../NativeReactNativeFeatureFlags.cpp | 7 +- .../NativeReactNativeFeatureFlags.h | 4 +- .../renderer/components/image/CMakeLists.txt | 1 - .../renderer/components/image/ImageProps.cpp | 68 +++--- .../components/scrollview/CMakeLists.txt | 1 - .../components/scrollview/ScrollViewProps.cpp | 163 ++++++------- .../components/text/BaseTextProps.cpp | 4 +- .../renderer/components/text/CMakeLists.txt | 1 - .../components/text/ParagraphProps.cpp | 36 +-- .../components/textinput/CMakeLists.txt | 1 - .../AndroidTextInputProps.cpp | 118 ++++----- .../components/view/AccessibilityProps.cpp | 66 +++--- .../components/view/BaseViewProps.cpp | 223 +++++++++--------- .../renderer/components/view/CMakeLists.txt | 1 - .../components/view/YogaStylableProps.cpp | 2 +- .../components/view/HostPlatformViewProps.cpp | 44 ++-- .../react/renderer/core/CMakeLists.txt | 1 - .../core/ConcreteComponentDescriptor.h | 3 +- .../ReactCommon/react/renderer/core/Props.cpp | 4 +- .../ReactCommon/react/utils/CoreFeatures.cpp | 1 + .../ReactCommon/react/utils/CoreFeatures.h | 3 + .../ReactNativeFeatureFlags.config.js | 5 - .../NativeReactNativeFeatureFlags.js | 3 +- .../featureflags/ReactNativeFeatureFlags.js | 7 +- 42 files changed, 387 insertions(+), 506 deletions(-) diff --git a/packages/react-native/React/Fabric/RCTSurfacePresenter.mm b/packages/react-native/React/Fabric/RCTSurfacePresenter.mm index 78a438c39e5ceb..4c1e048c4063f2 100644 --- a/packages/react-native/React/Fabric/RCTSurfacePresenter.mm +++ b/packages/react-native/React/Fabric/RCTSurfacePresenter.mm @@ -257,6 +257,10 @@ - (RCTScheduler *)_createScheduler { auto reactNativeConfig = _contextContainer->at>("ReactNativeConfig"); + if (reactNativeConfig && reactNativeConfig->getBool("react_fabric:enable_cpp_props_iterator_setter_ios")) { + CoreFeatures::enablePropIteratorSetter = true; + } + if (reactNativeConfig && reactNativeConfig->getBool("react_fabric:enable_granular_scroll_view_state_updates_ios")) { CoreFeatures::enableGranularScrollViewStateUpdatesIOS = true; } diff --git a/packages/react-native/ReactAndroid/api/ReactAndroid.api b/packages/react-native/ReactAndroid/api/ReactAndroid.api index 1cf4ea311e6f19..a8b0873a3631b5 100644 --- a/packages/react-native/ReactAndroid/api/ReactAndroid.api +++ b/packages/react-native/ReactAndroid/api/ReactAndroid.api @@ -1890,6 +1890,7 @@ public class com/facebook/react/config/ReactFeatureFlags { public static field enableBridgelessArchitectureNewCreateReloadDestroy Z public static field enableBridgelessArchitectureSoftExceptions Z public static field enableClonelessStateProgression Z + public static field enableCppPropsIteratorSetter Z public static field enableEagerRootViewAttachment Z public static field enableFabricLogs Z public static field enableFabricPendingEventQueue Z diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java index b8c94f4603781b..57faf8d7abff2d 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/config/ReactFeatureFlags.java @@ -99,6 +99,11 @@ public class ReactFeatureFlags { */ public static boolean enableViewRecycling = false; + /** + * Enable prop iterator setter-style construction of Props in C++ (this flag is not used in Java). + */ + public static boolean enableCppPropsIteratorSetter = false; + /** * Allow Differentiator.cpp and FabricMountingManager.cpp to generate a RemoveDeleteTree mega-op. */ diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlags.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlags.kt index 72346b0dd2e207..487f43d409c9a4 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlags.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlags.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<> */ /** @@ -88,12 +88,6 @@ public object ReactNativeFeatureFlags { @JvmStatic public fun inspectorEnableModernCDPRegistry(): Boolean = accessor.inspectorEnableModernCDPRegistry() - /** - * Specifies whether the iterator-style prop parsing is enabled. - */ - @JvmStatic - public fun enablePropIteratorSetter(): Boolean = accessor.enablePropIteratorSetter() - /** * Overrides the feature flags with the ones provided by the given provider * (generally one that extends `ReactNativeFeatureFlagsDefaults`). diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxAccessor.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxAccessor.kt index e7650dff5a9adc..3e38e9f251cbd8 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxAccessor.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxAccessor.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<144af28ad2eb5bf179ed003da3d76165>> + * @generated SignedSource<<11824621ee7ca5dbdf2f09bdf1a1f983>> */ /** @@ -30,7 +30,6 @@ public class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAccesso private var enableFixForClippedSubviewsCrashCache: Boolean? = null private var inspectorEnableCxxInspectorPackagerConnectionCache: Boolean? = null private var inspectorEnableModernCDPRegistryCache: Boolean? = null - private var enablePropIteratorSetterCache: Boolean? = null override fun commonTestFlag(): Boolean { var cached = commonTestFlagCache @@ -122,15 +121,6 @@ public class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAccesso return cached } - override fun enablePropIteratorSetter(): Boolean { - var cached = enablePropIteratorSetterCache - if (cached == null) { - cached = ReactNativeFeatureFlagsCxxInterop.enablePropIteratorSetter() - enablePropIteratorSetterCache = cached - } - return cached - } - override fun override(provider: ReactNativeFeatureFlagsProvider): Unit = ReactNativeFeatureFlagsCxxInterop.override(provider as Any) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxInterop.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxInterop.kt index 3f5e9ddd696879..d3518e6cba082d 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxInterop.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsCxxInterop.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<> */ /** @@ -48,8 +48,6 @@ public object ReactNativeFeatureFlagsCxxInterop { @DoNotStrip @JvmStatic public external fun inspectorEnableModernCDPRegistry(): Boolean - @DoNotStrip @JvmStatic public external fun enablePropIteratorSetter(): Boolean - @DoNotStrip @JvmStatic public external fun override(provider: Any) @DoNotStrip @JvmStatic public external fun dangerouslyReset() diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsDefaults.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsDefaults.kt index 0e11c667f9347f..e75ba65f44d08b 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsDefaults.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsDefaults.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<> */ /** @@ -42,6 +42,4 @@ public open class ReactNativeFeatureFlagsDefaults : ReactNativeFeatureFlagsProvi override fun inspectorEnableCxxInspectorPackagerConnection(): Boolean = false override fun inspectorEnableModernCDPRegistry(): Boolean = false - - override fun enablePropIteratorSetter(): Boolean = false } diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsLocalAccessor.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsLocalAccessor.kt index 88aee6e36b9b78..2fcc689a55e16c 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsLocalAccessor.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsLocalAccessor.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<<63356ad414e641eae11ca07b1a876fd3>> */ /** @@ -34,7 +34,6 @@ public class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcces private var enableFixForClippedSubviewsCrashCache: Boolean? = null private var inspectorEnableCxxInspectorPackagerConnectionCache: Boolean? = null private var inspectorEnableModernCDPRegistryCache: Boolean? = null - private var enablePropIteratorSetterCache: Boolean? = null override fun commonTestFlag(): Boolean { var cached = commonTestFlagCache @@ -136,16 +135,6 @@ public class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcces return cached } - override fun enablePropIteratorSetter(): Boolean { - var cached = enablePropIteratorSetterCache - if (cached == null) { - cached = currentProvider.enablePropIteratorSetter() - accessedFeatureFlags.add("enablePropIteratorSetter") - enablePropIteratorSetterCache = cached - } - return cached - } - override fun override(provider: ReactNativeFeatureFlagsProvider) { if (accessedFeatureFlags.isNotEmpty()) { val accessedFeatureFlagsStr = accessedFeatureFlags.joinToString(separator = ", ") { it } diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsProvider.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsProvider.kt index 403cb8128789a9..2d4561cc3e6420 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsProvider.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/featureflags/ReactNativeFeatureFlagsProvider.kt @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<01a82369c1bababe47535b8cceb487eb>> + * @generated SignedSource<> */ /** @@ -42,6 +42,4 @@ public interface ReactNativeFeatureFlagsProvider { @DoNotStrip public fun inspectorEnableCxxInspectorPackagerConnection(): Boolean @DoNotStrip public fun inspectorEnableModernCDPRegistry(): Boolean - - @DoNotStrip public fun enablePropIteratorSetter(): Boolean } diff --git a/packages/react-native/ReactAndroid/src/main/jni/react/fabric/Binding.cpp b/packages/react-native/ReactAndroid/src/main/jni/react/fabric/Binding.cpp index 368e6a2eb6e19f..b19acb9108ace0 100644 --- a/packages/react-native/ReactAndroid/src/main/jni/react/fabric/Binding.cpp +++ b/packages/react-native/ReactAndroid/src/main/jni/react/fabric/Binding.cpp @@ -413,6 +413,8 @@ void Binding::installFabricUIManager( "CalculateTransformedFramesEnabled", getFeatureFlagValue("calculateTransformedFramesEnabled")); + CoreFeatures::enablePropIteratorSetter = + getFeatureFlagValue("enableCppPropsIteratorSetter"); CoreFeatures::enableClonelessStateProgression = getFeatureFlagValue("enableClonelessStateProgression"); CoreFeatures::excludeYogaFromRawProps = diff --git a/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.cpp b/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.cpp index 3f7ec478c96a8a..15061428a2a2df 100644 --- a/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.cpp +++ b/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.cpp @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<> */ /** @@ -99,12 +99,6 @@ class ReactNativeFeatureFlagsProviderHolder return method(javaProvider_); } - bool enablePropIteratorSetter() override { - static const auto method = - getReactNativeFeatureFlagsProviderJavaClass()->getMethod("enablePropIteratorSetter"); - return method(javaProvider_); - } - private: jni::global_ref javaProvider_; }; @@ -159,11 +153,6 @@ bool JReactNativeFeatureFlagsCxxInterop::inspectorEnableModernCDPRegistry( return ReactNativeFeatureFlags::inspectorEnableModernCDPRegistry(); } -bool JReactNativeFeatureFlagsCxxInterop::enablePropIteratorSetter( - facebook::jni::alias_ref /*unused*/) { - return ReactNativeFeatureFlags::enablePropIteratorSetter(); -} - void JReactNativeFeatureFlagsCxxInterop::override( facebook::jni::alias_ref /*unused*/, jni::alias_ref provider) { @@ -211,9 +200,6 @@ void JReactNativeFeatureFlagsCxxInterop::registerNatives() { makeNativeMethod( "inspectorEnableModernCDPRegistry", JReactNativeFeatureFlagsCxxInterop::inspectorEnableModernCDPRegistry), - makeNativeMethod( - "enablePropIteratorSetter", - JReactNativeFeatureFlagsCxxInterop::enablePropIteratorSetter), }); } diff --git a/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.h b/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.h index 09af44eae5afc3..ff7881e09e417c 100644 --- a/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.h +++ b/packages/react-native/ReactAndroid/src/main/jni/react/featureflags/JReactNativeFeatureFlagsCxxInterop.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<70f4f8c1d6f86bcbcb13af782e7356cd>> + * @generated SignedSource<> */ /** @@ -60,9 +60,6 @@ class JReactNativeFeatureFlagsCxxInterop static bool inspectorEnableModernCDPRegistry( facebook::jni::alias_ref); - static bool enablePropIteratorSetter( - facebook::jni::alias_ref); - static void override( facebook::jni::alias_ref, jni::alias_ref provider); diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.cpp b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.cpp index 255e6f7cf6398f..caec5678dd7b3c 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.cpp +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.cpp @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<<82f226df2b3824d03b755a042b20bec5>> */ /** @@ -61,10 +61,6 @@ bool ReactNativeFeatureFlags::inspectorEnableModernCDPRegistry() { return getAccessor().inspectorEnableModernCDPRegistry(); } -bool ReactNativeFeatureFlags::enablePropIteratorSetter() { - return getAccessor().enablePropIteratorSetter(); -} - void ReactNativeFeatureFlags::override( std::unique_ptr provider) { getAccessor().override(std::move(provider)); diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.h b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.h index cddc4796c99bfd..8f82ea192014c9 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.h +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlags.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<98243d4cb18b03d4a2273416879f1081>> + * @generated SignedSource<<3bfae310dfd28040f4b80d1a1df8b7b3>> */ /** @@ -83,11 +83,6 @@ class ReactNativeFeatureFlags { */ static bool inspectorEnableModernCDPRegistry(); - /** - * Specifies whether the iterator-style prop parsing is enabled. - */ - static bool enablePropIteratorSetter(); - /** * Overrides the feature flags with the ones provided by the given provider * (generally one that extends `ReactNativeFeatureFlagsDefaults`). diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.cpp b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.cpp index 69081a4ca39406..d46d461bfd4261 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.cpp +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.cpp @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<94d6891851ba8d96e9049e9a4eb727ca>> + * @generated SignedSource<<2154292f89306f25289583537833a065>> */ /** @@ -209,24 +209,6 @@ bool ReactNativeFeatureFlagsAccessor::inspectorEnableModernCDPRegistry() { return flagValue.value(); } -bool ReactNativeFeatureFlagsAccessor::enablePropIteratorSetter() { - auto flagValue = enablePropIteratorSetter_.load(); - - if (!flagValue.has_value()) { - // This block is not exclusive but it is not necessary. - // If multiple threads try to initialize the feature flag, we would only - // be accessing the provider multiple times but the end state of this - // instance and the returned flag value would be the same. - - markFlagAsAccessed(10, "enablePropIteratorSetter"); - - flagValue = currentProvider_->enablePropIteratorSetter(); - enablePropIteratorSetter_ = flagValue; - } - - return flagValue.value(); -} - void ReactNativeFeatureFlagsAccessor::override( std::unique_ptr provider) { if (wasOverridden_) { diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.h b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.h index 8235f0f78e00f4..b177bfe53b6b33 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.h +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsAccessor.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<756f3b972dcc9f96e4fd5d0c67e945b6>> + * @generated SignedSource<<95487968b66d40e1ec53936b06084931>> */ /** @@ -41,7 +41,6 @@ class ReactNativeFeatureFlagsAccessor { bool enableFixForClippedSubviewsCrash(); bool inspectorEnableCxxInspectorPackagerConnection(); bool inspectorEnableModernCDPRegistry(); - bool enablePropIteratorSetter(); void override(std::unique_ptr provider); @@ -52,7 +51,7 @@ class ReactNativeFeatureFlagsAccessor { std::unique_ptr currentProvider_; bool wasOverridden_; - std::array, 11> accessedFeatureFlags_; + std::array, 10> accessedFeatureFlags_; std::atomic> commonTestFlag_; std::atomic> enableBackgroundExecutor_; @@ -64,7 +63,6 @@ class ReactNativeFeatureFlagsAccessor { std::atomic> enableFixForClippedSubviewsCrash_; std::atomic> inspectorEnableCxxInspectorPackagerConnection_; std::atomic> inspectorEnableModernCDPRegistry_; - std::atomic> enablePropIteratorSetter_; }; } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsDefaults.h b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsDefaults.h index 269b94e63e5b3a..ce2aadae3b63ce 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsDefaults.h +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsDefaults.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<22f3908006ed4979a639c7e374e6dc9a>> + * @generated SignedSource<<4832483bb3648380f2bb9312311f579c>> */ /** @@ -66,10 +66,6 @@ class ReactNativeFeatureFlagsDefaults : public ReactNativeFeatureFlagsProvider { bool inspectorEnableModernCDPRegistry() override { return false; } - - bool enablePropIteratorSetter() override { - return false; - } }; } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsProvider.h b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsProvider.h index dc0314183478ef..f6f55b5fc15e4a 100644 --- a/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsProvider.h +++ b/packages/react-native/ReactCommon/react/featureflags/ReactNativeFeatureFlagsProvider.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<37a53122381f8d5c2b953c609963e59b>> + * @generated SignedSource<> */ /** @@ -35,7 +35,6 @@ class ReactNativeFeatureFlagsProvider { virtual bool enableFixForClippedSubviewsCrash() = 0; virtual bool inspectorEnableCxxInspectorPackagerConnection() = 0; virtual bool inspectorEnableModernCDPRegistry() = 0; - virtual bool enablePropIteratorSetter() = 0; }; } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp index 183aca512ea84d..67f799f556c1ee 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp +++ b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<696e31d417b64aaf55f0b96b42706f88>> + * @generated SignedSource<<2466af777a1d69da2c3810e5a2d53a70>> */ /** @@ -85,9 +85,4 @@ bool NativeReactNativeFeatureFlags::inspectorEnableModernCDPRegistry( return ReactNativeFeatureFlags::inspectorEnableModernCDPRegistry(); } -bool NativeReactNativeFeatureFlags::enablePropIteratorSetter( - jsi::Runtime& /*runtime*/) { - return ReactNativeFeatureFlags::enablePropIteratorSetter(); -} - } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h index 683b456884ada7..203dfce52c07e1 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h +++ b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<97db360262a3bba9909ad3f16d01140d>> + * @generated SignedSource<<30481dc57cc7f389d1ed87d097caedda>> */ /** @@ -49,8 +49,6 @@ class NativeReactNativeFeatureFlags bool inspectorEnableCxxInspectorPackagerConnection(jsi::Runtime& runtime); bool inspectorEnableModernCDPRegistry(jsi::Runtime& runtime); - - bool enablePropIteratorSetter(jsi::Runtime& runtime); }; } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt index 408858d4e95e82..9dbe78d706dfa0 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt @@ -26,7 +26,6 @@ target_link_libraries(rrc_image jsi react_debug react_utils - react_featureflags react_render_core react_render_debug react_render_graphics diff --git a/packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.cpp index e0f826db7e2cb3..7f660904ae83e8 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.cpp @@ -5,10 +5,10 @@ * LICENSE file in the root directory of this source tree. */ -#include #include #include #include +#include namespace facebook::react { @@ -18,16 +18,15 @@ ImageProps::ImageProps( const RawProps& rawProps) : ViewProps(context, sourceProps, rawProps), sources( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.sources - : convertRawProp( - context, - rawProps, - "source", - sourceProps.sources, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.sources + : convertRawProp( + context, + rawProps, + "source", + sourceProps.sources, + {})), defaultSources( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.defaultSources : convertRawProp( context, @@ -36,7 +35,7 @@ ImageProps::ImageProps( sourceProps.defaultSources, {})), resizeMode( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.resizeMode : convertRawProp( context, @@ -45,34 +44,31 @@ ImageProps::ImageProps( sourceProps.resizeMode, ImageResizeMode::Stretch)), blurRadius( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.blurRadius - : convertRawProp( - context, - rawProps, - "blurRadius", - sourceProps.blurRadius, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.blurRadius + : convertRawProp( + context, + rawProps, + "blurRadius", + sourceProps.blurRadius, + {})), capInsets( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.capInsets - : convertRawProp( - context, - rawProps, - "capInsets", - sourceProps.capInsets, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.capInsets + : convertRawProp( + context, + rawProps, + "capInsets", + sourceProps.capInsets, + {})), tintColor( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.tintColor - : convertRawProp( - context, - rawProps, - "tintColor", - sourceProps.tintColor, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.tintColor + : convertRawProp( + context, + rawProps, + "tintColor", + sourceProps.tintColor, + {})), internal_analyticTag( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.internal_analyticTag : convertRawProp( context, diff --git a/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt index 3966bdcf2ab953..4e58b4d1c90c7c 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt @@ -25,7 +25,6 @@ target_link_libraries(rrc_scrollview glog_init jsi react_debug - react_featureflags react_render_core react_render_debug react_render_graphics diff --git a/packages/react-native/ReactCommon/react/renderer/components/scrollview/ScrollViewProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/scrollview/ScrollViewProps.cpp index b85d9e9f653896..4bf5b3c038c2fb 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/scrollview/ScrollViewProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/scrollview/ScrollViewProps.cpp @@ -7,10 +7,10 @@ #include "ScrollViewProps.h" -#include #include #include #include +#include #include @@ -22,7 +22,7 @@ ScrollViewProps::ScrollViewProps( const RawProps& rawProps) : ViewProps(context, sourceProps, rawProps), alwaysBounceHorizontal( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.alwaysBounceHorizontal : convertRawProp( context, @@ -31,7 +31,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.alwaysBounceHorizontal, {})), alwaysBounceVertical( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.alwaysBounceVertical : convertRawProp( context, @@ -40,25 +40,23 @@ ScrollViewProps::ScrollViewProps( sourceProps.alwaysBounceVertical, {})), bounces( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.bounces - : convertRawProp( - context, - rawProps, - "bounces", - sourceProps.bounces, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.bounces + : convertRawProp( + context, + rawProps, + "bounces", + sourceProps.bounces, + true)), bouncesZoom( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.bouncesZoom - : convertRawProp( - context, - rawProps, - "bouncesZoom", - sourceProps.bouncesZoom, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.bouncesZoom + : convertRawProp( + context, + rawProps, + "bouncesZoom", + sourceProps.bouncesZoom, + true)), canCancelContentTouches( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.canCancelContentTouches : convertRawProp( context, @@ -67,7 +65,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.canCancelContentTouches, true)), centerContent( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.centerContent : convertRawProp( context, @@ -76,7 +74,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.centerContent, {})), automaticallyAdjustContentInsets( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.automaticallyAdjustContentInsets : convertRawProp( context, @@ -85,7 +83,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.automaticallyAdjustContentInsets, {})), automaticallyAdjustsScrollIndicatorInsets( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.automaticallyAdjustsScrollIndicatorInsets : convertRawProp( context, @@ -94,7 +92,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.automaticallyAdjustsScrollIndicatorInsets, true)), decelerationRate( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.decelerationRate : convertRawProp( context, @@ -103,7 +101,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.decelerationRate, (Float)0.998)), directionalLockEnabled( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.directionalLockEnabled : convertRawProp( context, @@ -112,7 +110,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.directionalLockEnabled, {})), indicatorStyle( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.indicatorStyle : convertRawProp( context, @@ -121,7 +119,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.indicatorStyle, {})), keyboardDismissMode( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.keyboardDismissMode : convertRawProp( context, @@ -130,7 +128,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.keyboardDismissMode, {})), maintainVisibleContentPosition( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.maintainVisibleContentPosition : convertRawProp( context, @@ -139,7 +137,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.maintainVisibleContentPosition, {})), maximumZoomScale( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.maximumZoomScale : convertRawProp( context, @@ -148,7 +146,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.maximumZoomScale, (Float)1.0)), minimumZoomScale( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.minimumZoomScale : convertRawProp( context, @@ -157,7 +155,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.minimumZoomScale, (Float)1.0)), scrollEnabled( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.scrollEnabled : convertRawProp( context, @@ -166,7 +164,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.scrollEnabled, true)), pagingEnabled( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.pagingEnabled : convertRawProp( context, @@ -175,7 +173,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.pagingEnabled, {})), pinchGestureEnabled( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.pinchGestureEnabled : convertRawProp( context, @@ -184,16 +182,15 @@ ScrollViewProps::ScrollViewProps( sourceProps.pinchGestureEnabled, true)), scrollsToTop( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.scrollsToTop - : convertRawProp( - context, - rawProps, - "scrollsToTop", - sourceProps.scrollsToTop, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.scrollsToTop + : convertRawProp( + context, + rawProps, + "scrollsToTop", + sourceProps.scrollsToTop, + true)), showsHorizontalScrollIndicator( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.showsHorizontalScrollIndicator : convertRawProp( context, @@ -202,7 +199,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.showsHorizontalScrollIndicator, true)), showsVerticalScrollIndicator( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.showsVerticalScrollIndicator : convertRawProp( context, @@ -211,7 +208,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.showsVerticalScrollIndicator, true)), scrollEventThrottle( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.scrollEventThrottle : convertRawProp( context, @@ -220,25 +217,23 @@ ScrollViewProps::ScrollViewProps( sourceProps.scrollEventThrottle, {})), zoomScale( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.zoomScale - : convertRawProp( - context, - rawProps, - "zoomScale", - sourceProps.zoomScale, - (Float)1.0)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.zoomScale + : convertRawProp( + context, + rawProps, + "zoomScale", + sourceProps.zoomScale, + (Float)1.0)), contentInset( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.contentInset - : convertRawProp( - context, - rawProps, - "contentInset", - sourceProps.contentInset, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.contentInset + : convertRawProp( + context, + rawProps, + "contentInset", + sourceProps.contentInset, + {})), contentOffset( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.contentOffset : convertRawProp( context, @@ -247,7 +242,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.contentOffset, {})), scrollIndicatorInsets( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.scrollIndicatorInsets : convertRawProp( context, @@ -256,7 +251,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.scrollIndicatorInsets, {})), snapToInterval( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.snapToInterval : convertRawProp( context, @@ -265,7 +260,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.snapToInterval, {})), snapToAlignment( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.snapToAlignment : convertRawProp( context, @@ -274,7 +269,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.snapToAlignment, {})), disableIntervalMomentum( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.disableIntervalMomentum : convertRawProp( context, @@ -283,7 +278,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.disableIntervalMomentum, {})), snapToOffsets( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.snapToOffsets : convertRawProp( context, @@ -292,25 +287,23 @@ ScrollViewProps::ScrollViewProps( sourceProps.snapToOffsets, {})), snapToStart( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.snapToStart - : convertRawProp( - context, - rawProps, - "snapToStart", - sourceProps.snapToStart, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.snapToStart + : convertRawProp( + context, + rawProps, + "snapToStart", + sourceProps.snapToStart, + true)), snapToEnd( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.snapToEnd - : convertRawProp( - context, - rawProps, - "snapToEnd", - sourceProps.snapToEnd, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.snapToEnd + : convertRawProp( + context, + rawProps, + "snapToEnd", + sourceProps.snapToEnd, + true)), contentInsetAdjustmentBehavior( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.contentInsetAdjustmentBehavior : convertRawProp( context, @@ -319,7 +312,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.contentInsetAdjustmentBehavior, {ContentInsetAdjustmentBehavior::Never})), scrollToOverflowEnabled( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.scrollToOverflowEnabled : convertRawProp( context, @@ -328,7 +321,7 @@ ScrollViewProps::ScrollViewProps( sourceProps.scrollToOverflowEnabled, {})), isInvertedVirtualizedList( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.isInvertedVirtualizedList : convertRawProp( context, diff --git a/packages/react-native/ReactCommon/react/renderer/components/text/BaseTextProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/text/BaseTextProps.cpp index fce05ca8d450f6..cb5a93b703f87f 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/text/BaseTextProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/text/BaseTextProps.cpp @@ -7,11 +7,11 @@ #include "BaseTextProps.h" -#include #include #include #include #include +#include namespace facebook::react { @@ -218,7 +218,7 @@ BaseTextProps::BaseTextProps( const BaseTextProps& sourceProps, const RawProps& rawProps) : textAttributes( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.textAttributes : convertRawProp( context, diff --git a/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt index 1d5cbf7914a5d7..394d72d7de529b 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt @@ -25,7 +25,6 @@ target_link_libraries(rrc_text glog_init jsi react_debug - react_featureflags react_render_attributedstring react_render_core react_render_debug diff --git a/packages/react-native/ReactCommon/react/renderer/components/text/ParagraphProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/text/ParagraphProps.cpp index 173c822e3580dc..4011bbf2dc8f7e 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/text/ParagraphProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/text/ParagraphProps.cpp @@ -7,11 +7,13 @@ #include "ParagraphProps.h" -#include #include #include #include #include +#include + +#include namespace facebook::react { @@ -22,7 +24,7 @@ ParagraphProps::ParagraphProps( : ViewProps(context, sourceProps, rawProps), BaseTextProps(context, sourceProps, rawProps), paragraphAttributes( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.paragraphAttributes : convertRawProp( context, @@ -30,23 +32,21 @@ ParagraphProps::ParagraphProps( sourceProps.paragraphAttributes, {})), isSelectable( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.isSelectable - : convertRawProp( - context, - rawProps, - "selectable", - sourceProps.isSelectable, - false)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.isSelectable + : convertRawProp( + context, + rawProps, + "selectable", + sourceProps.isSelectable, + false)), onTextLayout( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.onTextLayout - : convertRawProp( - context, - rawProps, - "onTextLayout", - sourceProps.onTextLayout, - {})) { + CoreFeatures::enablePropIteratorSetter ? sourceProps.onTextLayout + : convertRawProp( + context, + rawProps, + "onTextLayout", + sourceProps.onTextLayout, + {})) { /* * These props are applied to `View`, therefore they must not be a part of * base text attributes. diff --git a/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt index 1ac61dd16eef55..e8d12d59df91fd 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt @@ -28,7 +28,6 @@ target_link_libraries(rrc_textinput glog_init jsi react_debug - react_featureflags react_render_attributedstring react_render_componentregistry react_render_core diff --git a/packages/react-native/ReactCommon/react/renderer/components/textinput/platform/android/react/renderer/components/androidtextinput/AndroidTextInputProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/textinput/platform/android/react/renderer/components/androidtextinput/AndroidTextInputProps.cpp index 2b587ad4cf3118..ea0fb6a1674e3d 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/textinput/platform/android/react/renderer/components/androidtextinput/AndroidTextInputProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/textinput/platform/android/react/renderer/components/androidtextinput/AndroidTextInputProps.cpp @@ -6,10 +6,10 @@ */ #include "AndroidTextInputProps.h" -#include #include #include #include +#include namespace facebook::react { @@ -38,209 +38,209 @@ AndroidTextInputProps::AndroidTextInputProps( const RawProps &rawProps) : ViewProps(context, sourceProps, rawProps), BaseTextProps(context, sourceProps, rawProps), - autoComplete(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.autoComplete : convertRawProp( + autoComplete(CoreFeatures::enablePropIteratorSetter? sourceProps.autoComplete : convertRawProp( context, rawProps, "autoComplete", sourceProps.autoComplete, {})), - returnKeyLabel(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.autoComplete : convertRawProp(context, rawProps, + returnKeyLabel(CoreFeatures::enablePropIteratorSetter? sourceProps.autoComplete : convertRawProp(context, rawProps, "returnKeyLabel", sourceProps.returnKeyLabel, {})), - numberOfLines(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.numberOfLines : convertRawProp(context, rawProps, + numberOfLines(CoreFeatures::enablePropIteratorSetter? sourceProps.numberOfLines : convertRawProp(context, rawProps, "numberOfLines", sourceProps.numberOfLines, {0})), - disableFullscreenUI(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.disableFullscreenUI : convertRawProp(context, rawProps, + disableFullscreenUI(CoreFeatures::enablePropIteratorSetter? sourceProps.disableFullscreenUI : convertRawProp(context, rawProps, "disableFullscreenUI", sourceProps.disableFullscreenUI, {false})), - textBreakStrategy(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textBreakStrategy : convertRawProp(context, rawProps, + textBreakStrategy(CoreFeatures::enablePropIteratorSetter? sourceProps.textBreakStrategy : convertRawProp(context, rawProps, "textBreakStrategy", sourceProps.textBreakStrategy, {})), - underlineColorAndroid(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.underlineColorAndroid : convertRawProp(context, rawProps, + underlineColorAndroid(CoreFeatures::enablePropIteratorSetter? sourceProps.underlineColorAndroid : convertRawProp(context, rawProps, "underlineColorAndroid", sourceProps.underlineColorAndroid, {})), - inlineImageLeft(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.inlineImageLeft : convertRawProp(context, rawProps, + inlineImageLeft(CoreFeatures::enablePropIteratorSetter? sourceProps.inlineImageLeft : convertRawProp(context, rawProps, "inlineImageLeft", sourceProps.inlineImageLeft, {})), - inlineImagePadding(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.inlineImagePadding : convertRawProp(context, rawProps, + inlineImagePadding(CoreFeatures::enablePropIteratorSetter? sourceProps.inlineImagePadding : convertRawProp(context, rawProps, "inlineImagePadding", sourceProps.inlineImagePadding, {0})), - importantForAutofill(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.importantForAutofill : convertRawProp(context, rawProps, + importantForAutofill(CoreFeatures::enablePropIteratorSetter? sourceProps.importantForAutofill : convertRawProp(context, rawProps, "importantForAutofill", sourceProps.importantForAutofill, {})), - showSoftInputOnFocus(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.showSoftInputOnFocus : convertRawProp(context, rawProps, + showSoftInputOnFocus(CoreFeatures::enablePropIteratorSetter? sourceProps.showSoftInputOnFocus : convertRawProp(context, rawProps, "showSoftInputOnFocus", sourceProps.showSoftInputOnFocus, {false})), - autoCapitalize(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.autoCapitalize : convertRawProp(context, rawProps, + autoCapitalize(CoreFeatures::enablePropIteratorSetter? sourceProps.autoCapitalize : convertRawProp(context, rawProps, "autoCapitalize", sourceProps.autoCapitalize, {})), - autoCorrect(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.autoCorrect : convertRawProp(context, rawProps, + autoCorrect(CoreFeatures::enablePropIteratorSetter? sourceProps.autoCorrect : convertRawProp(context, rawProps, "autoCorrect", sourceProps.autoCorrect, {false})), - autoFocus(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.autoFocus : convertRawProp(context, rawProps, + autoFocus(CoreFeatures::enablePropIteratorSetter? sourceProps.autoFocus : convertRawProp(context, rawProps, "autoFocus", sourceProps.autoFocus, {false})), - allowFontScaling(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.allowFontScaling : convertRawProp(context, rawProps, + allowFontScaling(CoreFeatures::enablePropIteratorSetter? sourceProps.allowFontScaling : convertRawProp(context, rawProps, "allowFontScaling", sourceProps.allowFontScaling, {false})), - maxFontSizeMultiplier(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.maxFontSizeMultiplier : convertRawProp(context, rawProps, + maxFontSizeMultiplier(CoreFeatures::enablePropIteratorSetter? sourceProps.maxFontSizeMultiplier : convertRawProp(context, rawProps, "maxFontSizeMultiplier", sourceProps.maxFontSizeMultiplier, {0.0})), - editable(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.editable : + editable(CoreFeatures::enablePropIteratorSetter? sourceProps.editable : convertRawProp(context, rawProps, "editable", sourceProps.editable, {false})), - keyboardType(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.keyboardType : convertRawProp(context, rawProps, + keyboardType(CoreFeatures::enablePropIteratorSetter? sourceProps.keyboardType : convertRawProp(context, rawProps, "keyboardType", sourceProps.keyboardType, {})), - returnKeyType(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.returnKeyType : convertRawProp(context, rawProps, + returnKeyType(CoreFeatures::enablePropIteratorSetter? sourceProps.returnKeyType : convertRawProp(context, rawProps, "returnKeyType", sourceProps.returnKeyType, {})), - maxLength(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.maxLength : + maxLength(CoreFeatures::enablePropIteratorSetter? sourceProps.maxLength : convertRawProp(context, rawProps, "maxLength", sourceProps.maxLength, {0})), - multiline(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.multiline : convertRawProp(context, rawProps, + multiline(CoreFeatures::enablePropIteratorSetter? sourceProps.multiline : convertRawProp(context, rawProps, "multiline", sourceProps.multiline, {false})), - placeholder(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.placeholder : + placeholder(CoreFeatures::enablePropIteratorSetter? sourceProps.placeholder : convertRawProp(context, rawProps, "placeholder", sourceProps.placeholder, {})), - placeholderTextColor(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.placeholderTextColor : convertRawProp(context, rawProps, + placeholderTextColor(CoreFeatures::enablePropIteratorSetter? sourceProps.placeholderTextColor : convertRawProp(context, rawProps, "placeholderTextColor", sourceProps.placeholderTextColor, {})), - secureTextEntry(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.secureTextEntry : convertRawProp(context, rawProps, + secureTextEntry(CoreFeatures::enablePropIteratorSetter? sourceProps.secureTextEntry : convertRawProp(context, rawProps, "secureTextEntry", sourceProps.secureTextEntry, {false})), - selectionColor(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.selectionColor : convertRawProp(context, rawProps, + selectionColor(CoreFeatures::enablePropIteratorSetter? sourceProps.selectionColor : convertRawProp(context, rawProps, "selectionColor", sourceProps.selectionColor, {})), - selectionHandleColor(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.selectionHandleColor : convertRawProp(context, rawProps, + selectionHandleColor(CoreFeatures::enablePropIteratorSetter? sourceProps.selectionHandleColor : convertRawProp(context, rawProps, "selectionHandleColor", sourceProps.selectionHandleColor, {})), - value(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.value : convertRawProp(context, rawProps, "value", sourceProps.value, {})), - defaultValue(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.defaultValue : convertRawProp(context, rawProps, + value(CoreFeatures::enablePropIteratorSetter? sourceProps.value : convertRawProp(context, rawProps, "value", sourceProps.value, {})), + defaultValue(CoreFeatures::enablePropIteratorSetter? sourceProps.defaultValue : convertRawProp(context, rawProps, "defaultValue", sourceProps.defaultValue, {})), - selectTextOnFocus(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.selectTextOnFocus : convertRawProp(context, rawProps, + selectTextOnFocus(CoreFeatures::enablePropIteratorSetter? sourceProps.selectTextOnFocus : convertRawProp(context, rawProps, "selectTextOnFocus", sourceProps.selectTextOnFocus, {false})), - submitBehavior(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.submitBehavior : convertRawProp(context, rawProps, + submitBehavior(CoreFeatures::enablePropIteratorSetter? sourceProps.submitBehavior : convertRawProp(context, rawProps, "submitBehavior", sourceProps.submitBehavior, {})), - caretHidden(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.caretHidden : convertRawProp(context, rawProps, + caretHidden(CoreFeatures::enablePropIteratorSetter? sourceProps.caretHidden : convertRawProp(context, rawProps, "caretHidden", sourceProps.caretHidden, {false})), - contextMenuHidden(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.contextMenuHidden : convertRawProp(context, rawProps, + contextMenuHidden(CoreFeatures::enablePropIteratorSetter? sourceProps.contextMenuHidden : convertRawProp(context, rawProps, "contextMenuHidden", sourceProps.contextMenuHidden, {false})), - textShadowColor(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textShadowColor : convertRawProp(context, rawProps, + textShadowColor(CoreFeatures::enablePropIteratorSetter? sourceProps.textShadowColor : convertRawProp(context, rawProps, "textShadowColor", sourceProps.textShadowColor, {})), - textShadowRadius(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textShadowRadius : convertRawProp(context, rawProps, + textShadowRadius(CoreFeatures::enablePropIteratorSetter? sourceProps.textShadowRadius : convertRawProp(context, rawProps, "textShadowRadius", sourceProps.textShadowRadius, {0.0})), - textDecorationLine(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textDecorationLine : convertRawProp(context, rawProps, + textDecorationLine(CoreFeatures::enablePropIteratorSetter? sourceProps.textDecorationLine : convertRawProp(context, rawProps, "textDecorationLine", sourceProps.textDecorationLine, {})), - fontStyle(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.fontStyle : + fontStyle(CoreFeatures::enablePropIteratorSetter? sourceProps.fontStyle : convertRawProp(context, rawProps, "fontStyle", sourceProps.fontStyle, {})), - textShadowOffset(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textShadowOffset : convertRawProp(context, rawProps, + textShadowOffset(CoreFeatures::enablePropIteratorSetter? sourceProps.textShadowOffset : convertRawProp(context, rawProps, "textShadowOffset", sourceProps.textShadowOffset, {})), - lineHeight(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.lineHeight : convertRawProp(context, rawProps, + lineHeight(CoreFeatures::enablePropIteratorSetter? sourceProps.lineHeight : convertRawProp(context, rawProps, "lineHeight", sourceProps.lineHeight, {0.0})), - textTransform(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textTransform : convertRawProp(context, rawProps, + textTransform(CoreFeatures::enablePropIteratorSetter? sourceProps.textTransform : convertRawProp(context, rawProps, "textTransform", sourceProps.textTransform, {})), color(0 /*convertRawProp(context, rawProps, "color", sourceProps.color, {0})*/), - letterSpacing(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.letterSpacing : convertRawProp(context, rawProps, + letterSpacing(CoreFeatures::enablePropIteratorSetter? sourceProps.letterSpacing : convertRawProp(context, rawProps, "letterSpacing", sourceProps.letterSpacing, {0.0})), - fontSize(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.fontSize : + fontSize(CoreFeatures::enablePropIteratorSetter? sourceProps.fontSize : convertRawProp(context, rawProps, "fontSize", sourceProps.fontSize, {0.0})), - textAlign(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textAlign : + textAlign(CoreFeatures::enablePropIteratorSetter? sourceProps.textAlign : convertRawProp(context, rawProps, "textAlign", sourceProps.textAlign, {})), - includeFontPadding(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.includeFontPadding : convertRawProp(context, rawProps, + includeFontPadding(CoreFeatures::enablePropIteratorSetter? sourceProps.includeFontPadding : convertRawProp(context, rawProps, "includeFontPadding", sourceProps.includeFontPadding, {false})), - fontWeight(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.fontWeight : + fontWeight(CoreFeatures::enablePropIteratorSetter? sourceProps.fontWeight : convertRawProp(context, rawProps, "fontWeight", sourceProps.fontWeight, {})), - fontFamily(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.fontFamily : + fontFamily(CoreFeatures::enablePropIteratorSetter? sourceProps.fontFamily : convertRawProp(context, rawProps, "fontFamily", sourceProps.fontFamily, {})), - textAlignVertical(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.textAlignVertical : convertRawProp(context, rawProps, + textAlignVertical(CoreFeatures::enablePropIteratorSetter? sourceProps.textAlignVertical : convertRawProp(context, rawProps, "textAlignVertical", sourceProps.textAlignVertical, {})), - cursorColor(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.cursorColor : + cursorColor(CoreFeatures::enablePropIteratorSetter? sourceProps.cursorColor : convertRawProp(context, rawProps, "cursorColor", sourceProps.cursorColor, {})), - mostRecentEventCount(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.mostRecentEventCount : convertRawProp(context, rawProps, + mostRecentEventCount(CoreFeatures::enablePropIteratorSetter? sourceProps.mostRecentEventCount : convertRawProp(context, rawProps, "mostRecentEventCount", sourceProps.mostRecentEventCount, {0})), - text(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.text : convertRawProp(context, rawProps, "text", sourceProps.text, {})), - paragraphAttributes(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.paragraphAttributes : + text(CoreFeatures::enablePropIteratorSetter? sourceProps.text : convertRawProp(context, rawProps, "text", sourceProps.text, {})), + paragraphAttributes(CoreFeatures::enablePropIteratorSetter? sourceProps.paragraphAttributes : convertRawProp(context, rawProps, sourceProps.paragraphAttributes, {})), // See AndroidTextInputComponentDescriptor for usage // TODO T63008435: can these, and this feature, be removed entirely? - hasPadding(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPadding : hasValue(rawProps, sourceProps.hasPadding, "padding")), - hasPaddingHorizontal(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingHorizontal : hasValue( + hasPadding(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPadding : hasValue(rawProps, sourceProps.hasPadding, "padding")), + hasPaddingHorizontal(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingHorizontal : hasValue( rawProps, sourceProps.hasPaddingHorizontal, "paddingHorizontal")), - hasPaddingVertical(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingVertical : hasValue( + hasPaddingVertical(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingVertical : hasValue( rawProps, sourceProps.hasPaddingVertical, "paddingVertical")), - hasPaddingLeft(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingLeft : hasValue( + hasPaddingLeft(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingLeft : hasValue( rawProps, sourceProps.hasPaddingLeft, "paddingLeft")), - hasPaddingTop(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingTop : + hasPaddingTop(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingTop : hasValue(rawProps, sourceProps.hasPaddingTop, "paddingTop")), - hasPaddingRight(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingRight : hasValue( + hasPaddingRight(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingRight : hasValue( rawProps, sourceProps.hasPaddingRight, "paddingRight")), - hasPaddingBottom(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingBottom : hasValue( + hasPaddingBottom(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingBottom : hasValue( rawProps, sourceProps.hasPaddingBottom, "paddingBottom")), - hasPaddingStart(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingStart : hasValue( + hasPaddingStart(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingStart : hasValue( rawProps, sourceProps.hasPaddingStart, "paddingStart")), - hasPaddingEnd(ReactNativeFeatureFlags::enablePropIteratorSetter()? sourceProps.hasPaddingEnd : + hasPaddingEnd(CoreFeatures::enablePropIteratorSetter? sourceProps.hasPaddingEnd : hasValue(rawProps, sourceProps.hasPaddingEnd, "paddingEnd")) { } diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/AccessibilityProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/view/AccessibilityProps.cpp index 13a9b28eb04de3..bb5e7a6f288acc 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/AccessibilityProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/view/AccessibilityProps.cpp @@ -7,11 +7,11 @@ #include "AccessibilityProps.h" -#include #include #include #include #include +#include namespace facebook::react { @@ -20,16 +20,15 @@ AccessibilityProps::AccessibilityProps( const AccessibilityProps& sourceProps, const RawProps& rawProps) : accessible( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.accessible - : convertRawProp( - context, - rawProps, - "accessible", - sourceProps.accessible, - false)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessible + : convertRawProp( + context, + rawProps, + "accessible", + sourceProps.accessible, + false)), accessibilityState( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityState : convertRawProp( context, @@ -38,7 +37,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityState, {})), accessibilityLabel( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityLabel : convertRawProp( context, @@ -47,7 +46,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityLabel, "")), accessibilityLabelledBy( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityLabelledBy : convertRawProp( context, @@ -56,7 +55,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityLabelledBy, {})), accessibilityLiveRegion( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityLiveRegion : convertRawProp( context, @@ -65,7 +64,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityLiveRegion, AccessibilityLiveRegion::None)), accessibilityHint( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityHint : convertRawProp( context, @@ -74,7 +73,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityHint, "")), accessibilityLanguage( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityLanguage : convertRawProp( context, @@ -83,7 +82,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityLanguage, "")), accessibilityValue( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityValue : convertRawProp( context, @@ -92,7 +91,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityValue, {})), accessibilityActions( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityActions : convertRawProp( context, @@ -101,7 +100,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityActions, {})), accessibilityViewIsModal( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityViewIsModal : convertRawProp( context, @@ -110,7 +109,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityViewIsModal, false)), accessibilityElementsHidden( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityElementsHidden : convertRawProp( context, @@ -119,7 +118,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityElementsHidden, false)), accessibilityIgnoresInvertColors( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.accessibilityIgnoresInvertColors : convertRawProp( context, @@ -128,7 +127,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.accessibilityIgnoresInvertColors, false)), onAccessibilityTap( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.onAccessibilityTap : convertRawProp( context, @@ -137,7 +136,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.onAccessibilityTap, {})), onAccessibilityMagicTap( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.onAccessibilityMagicTap : convertRawProp( context, @@ -146,7 +145,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.onAccessibilityMagicTap, {})), onAccessibilityEscape( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.onAccessibilityEscape : convertRawProp( context, @@ -155,7 +154,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.onAccessibilityEscape, {})), onAccessibilityAction( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.onAccessibilityAction : convertRawProp( context, @@ -164,7 +163,7 @@ AccessibilityProps::AccessibilityProps( sourceProps.onAccessibilityAction, {})), importantForAccessibility( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.importantForAccessibility : convertRawProp( context, @@ -173,14 +172,13 @@ AccessibilityProps::AccessibilityProps( sourceProps.importantForAccessibility, ImportantForAccessibility::Auto)), testId( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.testId - : convertRawProp( - context, - rawProps, - "testID", - sourceProps.testId, - "")) { + CoreFeatures::enablePropIteratorSetter ? sourceProps.testId + : convertRawProp( + context, + rawProps, + "testID", + sourceProps.testId, + "")) { // It is a (severe!) perf deoptimization to request props out-of-order. // Thus, since we need to request the same prop twice here // (accessibilityRole) we "must" do them subsequently here to prevent @@ -188,7 +186,7 @@ AccessibilityProps::AccessibilityProps( // it probably can, but this is a fairly rare edge-case that (1) is easy-ish // to work around here, and (2) would require very careful work to address // this case and not regress the more common cases. - if (!ReactNativeFeatureFlags::enablePropIteratorSetter()) { + if (!CoreFeatures::enablePropIteratorSetter) { auto* accessibilityRoleValue = rawProps.at("accessibilityRole", nullptr, nullptr); auto* roleValue = rawProps.at("role", nullptr, nullptr); diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.cpp index 71b6d7cacae4df..3d55f25c38cb9c 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.cpp @@ -9,12 +9,12 @@ #include -#include #include #include #include #include #include +#include namespace facebook::react { @@ -55,16 +55,15 @@ BaseViewProps::BaseViewProps( : YogaStylableProps(context, sourceProps, rawProps), AccessibilityProps(context, sourceProps, rawProps), opacity( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.opacity - : convertRawProp( - context, - rawProps, - "opacity", - sourceProps.opacity, - (Float)1.0)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.opacity + : convertRawProp( + context, + rawProps, + "opacity", + sourceProps.opacity, + (Float)1.0)), backgroundColor( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.backgroundColor : convertRawProp( context, @@ -73,65 +72,59 @@ BaseViewProps::BaseViewProps( sourceProps.backgroundColor, {})), borderRadii( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.borderRadii - : convertRawProp( - context, - rawProps, - "border", - "Radius", - sourceProps.borderRadii, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.borderRadii + : convertRawProp( + context, + rawProps, + "border", + "Radius", + sourceProps.borderRadii, + {})), borderColors( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.borderColors - : convertRawProp( - context, - rawProps, - "border", - "Color", - sourceProps.borderColors, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.borderColors + : convertRawProp( + context, + rawProps, + "border", + "Color", + sourceProps.borderColors, + {})), borderCurves( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.borderCurves - : convertRawProp( - context, - rawProps, - "border", - "Curve", - sourceProps.borderCurves, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.borderCurves + : convertRawProp( + context, + rawProps, + "border", + "Curve", + sourceProps.borderCurves, + {})), borderStyles( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.borderStyles - : convertRawProp( - context, - rawProps, - "border", - "Style", - sourceProps.borderStyles, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.borderStyles + : convertRawProp( + context, + rawProps, + "border", + "Style", + sourceProps.borderStyles, + {})), shadowColor( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.shadowColor - : convertRawProp( - context, - rawProps, - "shadowColor", - sourceProps.shadowColor, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.shadowColor + : convertRawProp( + context, + rawProps, + "shadowColor", + sourceProps.shadowColor, + {})), shadowOffset( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.shadowOffset - : convertRawProp( - context, - rawProps, - "shadowOffset", - sourceProps.shadowOffset, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.shadowOffset + : convertRawProp( + context, + rawProps, + "shadowOffset", + sourceProps.shadowOffset, + {})), shadowOpacity( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.shadowOpacity : convertRawProp( context, @@ -140,25 +133,23 @@ BaseViewProps::BaseViewProps( sourceProps.shadowOpacity, {})), shadowRadius( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.shadowRadius - : convertRawProp( - context, - rawProps, - "shadowRadius", - sourceProps.shadowRadius, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.shadowRadius + : convertRawProp( + context, + rawProps, + "shadowRadius", + sourceProps.shadowRadius, + {})), transform( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.transform - : convertRawProp( - context, - rawProps, - "transform", - sourceProps.transform, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.transform + : convertRawProp( + context, + rawProps, + "transform", + sourceProps.transform, + {})), transformOrigin( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.transformOrigin : convertRawProp( context, @@ -167,7 +158,7 @@ BaseViewProps::BaseViewProps( sourceProps.transformOrigin, {})), backfaceVisibility( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.backfaceVisibility : convertRawProp( context, @@ -176,7 +167,7 @@ BaseViewProps::BaseViewProps( sourceProps.backfaceVisibility, {})), shouldRasterize( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.shouldRasterize : convertRawProp( context, @@ -185,16 +176,15 @@ BaseViewProps::BaseViewProps( sourceProps.shouldRasterize, {})), zIndex( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.zIndex - : convertRawProp( - context, - rawProps, - "zIndex", - sourceProps.zIndex, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.zIndex + : convertRawProp( + context, + rawProps, + "zIndex", + sourceProps.zIndex, + {})), pointerEvents( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.pointerEvents : convertRawProp( context, @@ -203,38 +193,35 @@ BaseViewProps::BaseViewProps( sourceProps.pointerEvents, {})), hitSlop( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.hitSlop - : convertRawProp( - context, - rawProps, - "hitSlop", - sourceProps.hitSlop, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.hitSlop + : convertRawProp( + context, + rawProps, + "hitSlop", + sourceProps.hitSlop, + {})), onLayout( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.onLayout - : convertRawProp( - context, - rawProps, - "onLayout", - sourceProps.onLayout, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.onLayout + : convertRawProp( + context, + rawProps, + "onLayout", + sourceProps.onLayout, + {})), events( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.events : convertRawProp(context, rawProps, sourceProps.events, {})), collapsable( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.collapsable - : convertRawProp( - context, - rawProps, - "collapsable", - sourceProps.collapsable, - true)), + CoreFeatures::enablePropIteratorSetter ? sourceProps.collapsable + : convertRawProp( + context, + rawProps, + "collapsable", + sourceProps.collapsable, + true)), removeClippedSubviews( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.removeClippedSubviews : convertRawProp( context, @@ -243,7 +230,7 @@ BaseViewProps::BaseViewProps( sourceProps.removeClippedSubviews, false)), experimental_layoutConformance( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.experimental_layoutConformance : convertRawProp( context, diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt index cdff5d2f51c400..2cf299bcbb4bcc 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt @@ -33,7 +33,6 @@ target_link_libraries(rrc_view jsi logger react_debug - react_featureflags react_render_core react_render_debug react_render_graphics diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/YogaStylableProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/view/YogaStylableProps.cpp index d40bbdf66ad886..508e3aeb6cac0a 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/YogaStylableProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/view/YogaStylableProps.cpp @@ -7,11 +7,11 @@ #include "YogaStylableProps.h" -#include #include #include #include #include +#include #include namespace facebook::react { diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp b/packages/react-native/ReactCommon/react/renderer/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp index 7251b9caa56d1d..021268bfdac650 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp @@ -7,11 +7,13 @@ #include "HostPlatformViewProps.h" -#include +#include + #include #include #include #include +#include namespace facebook::react { @@ -21,16 +23,15 @@ HostPlatformViewProps::HostPlatformViewProps( const RawProps& rawProps) : BaseViewProps(context, sourceProps, rawProps), elevation( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.elevation - : convertRawProp( - context, - rawProps, - "elevation", - sourceProps.elevation, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.elevation + : convertRawProp( + context, + rawProps, + "elevation", + sourceProps.elevation, + {})), nativeBackground( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.nativeBackground : convertRawProp( context, @@ -39,7 +40,7 @@ HostPlatformViewProps::HostPlatformViewProps( sourceProps.nativeBackground, {})), nativeForeground( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.nativeForeground : convertRawProp( context, @@ -48,16 +49,15 @@ HostPlatformViewProps::HostPlatformViewProps( sourceProps.nativeForeground, {})), focusable( - ReactNativeFeatureFlags::enablePropIteratorSetter() - ? sourceProps.focusable - : convertRawProp( - context, - rawProps, - "focusable", - sourceProps.focusable, - {})), + CoreFeatures::enablePropIteratorSetter ? sourceProps.focusable + : convertRawProp( + context, + rawProps, + "focusable", + sourceProps.focusable, + {})), hasTVPreferredFocus( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.hasTVPreferredFocus : convertRawProp( context, @@ -66,7 +66,7 @@ HostPlatformViewProps::HostPlatformViewProps( sourceProps.hasTVPreferredFocus, {})), needsOffscreenAlphaCompositing( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.needsOffscreenAlphaCompositing : convertRawProp( context, @@ -75,7 +75,7 @@ HostPlatformViewProps::HostPlatformViewProps( sourceProps.needsOffscreenAlphaCompositing, {})), renderToHardwareTextureAndroid( - ReactNativeFeatureFlags::enablePropIteratorSetter() + CoreFeatures::enablePropIteratorSetter ? sourceProps.renderToHardwareTextureAndroid : convertRawProp( context, diff --git a/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt index bb8043d541a85f..7e24d29f97bb45 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt @@ -25,7 +25,6 @@ target_link_libraries(react_render_core logger react_config react_debug - react_featureflags react_render_debug react_render_graphics react_render_mapbuffer diff --git a/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h b/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h index 877d2068ad57ac..653f7700efa2a9 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h +++ b/packages/react-native/ReactCommon/react/renderer/core/ConcreteComponentDescriptor.h @@ -11,7 +11,6 @@ #include #include -#include #include #include #include @@ -118,7 +117,7 @@ class ConcreteComponentDescriptor : public ComponentDescriptor { // Use the new-style iterator // Note that we just check if `Props` has this flag set, no matter // the type of ShadowNode; it acts as the single global flag. - if (ReactNativeFeatureFlags::enablePropIteratorSetter()) { + if (CoreFeatures::enablePropIteratorSetter) { rawProps.iterateOverValues([&](RawPropsPropNameHash hash, const char* propName, const RawValue& fn) { diff --git a/packages/react-native/ReactCommon/react/renderer/core/Props.cpp b/packages/react-native/ReactCommon/react/renderer/core/Props.cpp index 3f19610cac9dc2..df133e8078e501 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/Props.cpp +++ b/packages/react-native/ReactCommon/react/renderer/core/Props.cpp @@ -8,8 +8,8 @@ #include "Props.h" #include -#include #include +#include namespace facebook::react { @@ -24,7 +24,7 @@ void Props::initialize( const PropsParserContext& context, const Props& sourceProps, const RawProps& rawProps) { - nativeId = ReactNativeFeatureFlags::enablePropIteratorSetter() + nativeId = CoreFeatures::enablePropIteratorSetter ? sourceProps.nativeId : convertRawProp(context, rawProps, "nativeID", sourceProps.nativeId, {}); #ifdef ANDROID diff --git a/packages/react-native/ReactCommon/react/utils/CoreFeatures.cpp b/packages/react-native/ReactCommon/react/utils/CoreFeatures.cpp index a12f55320965cb..1ae6897bfbc1cc 100644 --- a/packages/react-native/ReactCommon/react/utils/CoreFeatures.cpp +++ b/packages/react-native/ReactCommon/react/utils/CoreFeatures.cpp @@ -9,6 +9,7 @@ namespace facebook::react { +bool CoreFeatures::enablePropIteratorSetter = false; bool CoreFeatures::cacheLastTextMeasurement = false; bool CoreFeatures::enableGranularScrollViewStateUpdatesIOS = false; bool CoreFeatures::enableMountHooks = false; diff --git a/packages/react-native/ReactCommon/react/utils/CoreFeatures.h b/packages/react-native/ReactCommon/react/utils/CoreFeatures.h index da53aa052e7d9d..1cf5dc68d1e2df 100644 --- a/packages/react-native/ReactCommon/react/utils/CoreFeatures.h +++ b/packages/react-native/ReactCommon/react/utils/CoreFeatures.h @@ -16,6 +16,9 @@ namespace facebook::react { */ class CoreFeatures { public: + // Specifies whether the iterator-style prop parsing is enabled. + static bool enablePropIteratorSetter; + // Yoga might measure multiple times the same Text with the same constraints // This flag enables a caching mechanism to avoid subsequents measurements // of the same Text with the same constrainst. diff --git a/packages/react-native/scripts/featureflags/ReactNativeFeatureFlags.config.js b/packages/react-native/scripts/featureflags/ReactNativeFeatureFlags.config.js index fd034064433aa6..ef91f0f6e352c5 100644 --- a/packages/react-native/scripts/featureflags/ReactNativeFeatureFlags.config.js +++ b/packages/react-native/scripts/featureflags/ReactNativeFeatureFlags.config.js @@ -63,11 +63,6 @@ const definitions: FeatureFlagDefinitions = { 'Flag determining if the modern CDP backend should be enabled. This flag is global and should not be changed across React Host lifetimes.', defaultValue: false, }, - enablePropIteratorSetter: { - description: - 'Specifies whether the iterator-style prop parsing is enabled.', - defaultValue: false, - }, }, jsOnly: { diff --git a/packages/react-native/src/private/featureflags/NativeReactNativeFeatureFlags.js b/packages/react-native/src/private/featureflags/NativeReactNativeFeatureFlags.js index d537f8f2c64a33..93243673eba9d2 100644 --- a/packages/react-native/src/private/featureflags/NativeReactNativeFeatureFlags.js +++ b/packages/react-native/src/private/featureflags/NativeReactNativeFeatureFlags.js @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<1bf5b6cc6e759b02ab3dafd28f98db2c>> + * @generated SignedSource<<8509d5ee87efb5aa8da7efcd2085d0a2>> * @flow strict-local */ @@ -33,7 +33,6 @@ export interface Spec extends TurboModule { +enableFixForClippedSubviewsCrash?: () => boolean; +inspectorEnableCxxInspectorPackagerConnection?: () => boolean; +inspectorEnableModernCDPRegistry?: () => boolean; - +enablePropIteratorSetter?: () => boolean; } const NativeReactNativeFeatureFlags: ?Spec = TurboModuleRegistry.get( diff --git a/packages/react-native/src/private/featureflags/ReactNativeFeatureFlags.js b/packages/react-native/src/private/featureflags/ReactNativeFeatureFlags.js index bf3a5e906f3790..214dc80c030845 100644 --- a/packages/react-native/src/private/featureflags/ReactNativeFeatureFlags.js +++ b/packages/react-native/src/private/featureflags/ReactNativeFeatureFlags.js @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<702649b31fb7da1880fcc4e2b8b8e656>> + * @generated SignedSource<<7c83d5613c3be517efe48378e6356e79>> * @flow strict-local */ @@ -50,7 +50,6 @@ export type ReactNativeFeatureFlags = { enableFixForClippedSubviewsCrash: Getter, inspectorEnableCxxInspectorPackagerConnection: Getter, inspectorEnableModernCDPRegistry: Getter, - enablePropIteratorSetter: Getter, } /** @@ -133,10 +132,6 @@ export const inspectorEnableCxxInspectorPackagerConnection: Getter = cr * Flag determining if the modern CDP backend should be enabled. This flag is global and should not be changed across React Host lifetimes. */ export const inspectorEnableModernCDPRegistry: Getter = createNativeFlagGetter('inspectorEnableModernCDPRegistry', false); -/** - * Specifies whether the iterator-style prop parsing is enabled. - */ -export const enablePropIteratorSetter: Getter = createNativeFlagGetter('enablePropIteratorSetter', false); /** * Overrides the feature flags with the provided methods.