Skip to content

Commit 913e869

Browse files
javachemeta-codesync[bot]
authored andcommitted
Cleanup useNativeTransformHelperAndroid flag (#55031)
Summary: Pull Request resolved: #55031 This has been fully rolled out now, the feature flag can be removed. Changelog: [Internal] Reviewed By: cortinico Differential Revision: D90105012 fbshipit-source-id: c52516e88623259589be88c903fc287047a78e01
1 parent 492df85 commit 913e869

21 files changed

+31
-153
lines changed

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<<7fe8e1af0448cfb6857591608f2082ef>>
7+
* @generated SignedSource<<9406085adbc0685b473c868a27085cf8>>
88
*/
99

1010
/**
@@ -498,12 +498,6 @@ public object ReactNativeFeatureFlags {
498498
@JvmStatic
499499
public fun useFabricInterop(): Boolean = accessor.useFabricInterop()
500500

501-
/**
502-
* Use a native implementation of TransformHelper
503-
*/
504-
@JvmStatic
505-
public fun useNativeTransformHelperAndroid(): Boolean = accessor.useNativeTransformHelperAndroid()
506-
507501
/**
508502
* When enabled, the native view configs are used in bridgeless mode.
509503
*/

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<<046cc58eb9c6398863e6f58fc336d21f>>
7+
* @generated SignedSource<<accc9aa83d8ea2f5faa1eaae9a696d67>>
88
*/
99

1010
/**
@@ -98,7 +98,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
9898
private var updateRuntimeShadowNodeReferencesOnCommitCache: Boolean? = null
9999
private var useAlwaysAvailableJSErrorHandlingCache: Boolean? = null
100100
private var useFabricInteropCache: Boolean? = null
101-
private var useNativeTransformHelperAndroidCache: Boolean? = null
102101
private var useNativeViewConfigsInBridgelessModeCache: Boolean? = null
103102
private var useRawPropsJsiValueCache: Boolean? = null
104103
private var useShadowNodeStateOnCloneCache: Boolean? = null
@@ -812,15 +811,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
812811
return cached
813812
}
814813

815-
override fun useNativeTransformHelperAndroid(): Boolean {
816-
var cached = useNativeTransformHelperAndroidCache
817-
if (cached == null) {
818-
cached = ReactNativeFeatureFlagsCxxInterop.useNativeTransformHelperAndroid()
819-
useNativeTransformHelperAndroidCache = cached
820-
}
821-
return cached
822-
}
823-
824814
override fun useNativeViewConfigsInBridgelessMode(): Boolean {
825815
var cached = useNativeViewConfigsInBridgelessModeCache
826816
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<<5a12c4720aaf3b6c9ff010a5fadf9ea4>>
7+
* @generated SignedSource<<e3e8ad599dfc9c77d6a79d6c094e8a2b>>
88
*/
99

1010
/**
@@ -184,8 +184,6 @@ public object ReactNativeFeatureFlagsCxxInterop {
184184

185185
@DoNotStrip @JvmStatic public external fun useFabricInterop(): Boolean
186186

187-
@DoNotStrip @JvmStatic public external fun useNativeTransformHelperAndroid(): Boolean
188-
189187
@DoNotStrip @JvmStatic public external fun useNativeViewConfigsInBridgelessMode(): Boolean
190188

191189
@DoNotStrip @JvmStatic public external fun useRawPropsJsiValue(): 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<<8ff8090212f1d161d049cc02216bde1a>>
7+
* @generated SignedSource<<6bdc23594dd175a0bf4409c0e2f10552>>
88
*/
99

1010
/**
@@ -179,8 +179,6 @@ public open class ReactNativeFeatureFlagsDefaults : ReactNativeFeatureFlagsProvi
179179

180180
override fun useFabricInterop(): Boolean = true
181181

182-
override fun useNativeTransformHelperAndroid(): Boolean = true
183-
184182
override fun useNativeViewConfigsInBridgelessMode(): Boolean = false
185183

186184
override fun useRawPropsJsiValue(): 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<<04b0480c4675c067c183280370dd861a>>
7+
* @generated SignedSource<<3b0c2dec02a603afec82993302ecb6ff>>
88
*/
99

1010
/**
@@ -102,7 +102,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
102102
private var updateRuntimeShadowNodeReferencesOnCommitCache: Boolean? = null
103103
private var useAlwaysAvailableJSErrorHandlingCache: Boolean? = null
104104
private var useFabricInteropCache: Boolean? = null
105-
private var useNativeTransformHelperAndroidCache: Boolean? = null
106105
private var useNativeViewConfigsInBridgelessModeCache: Boolean? = null
107106
private var useRawPropsJsiValueCache: Boolean? = null
108107
private var useShadowNodeStateOnCloneCache: Boolean? = null
@@ -894,16 +893,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
894893
return cached
895894
}
896895

897-
override fun useNativeTransformHelperAndroid(): Boolean {
898-
var cached = useNativeTransformHelperAndroidCache
899-
if (cached == null) {
900-
cached = currentProvider.useNativeTransformHelperAndroid()
901-
accessedFeatureFlags.add("useNativeTransformHelperAndroid")
902-
useNativeTransformHelperAndroidCache = cached
903-
}
904-
return cached
905-
}
906-
907896
override fun useNativeViewConfigsInBridgelessMode(): Boolean {
908897
var cached = useNativeViewConfigsInBridgelessModeCache
909898
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<<f3df6f3fd411a0ddc17c1f3a0147e276>>
7+
* @generated SignedSource<<995c1c3035757935dc7a4c4d6778e43e>>
88
*/
99

1010
/**
@@ -179,8 +179,6 @@ public interface ReactNativeFeatureFlagsProvider {
179179

180180
@DoNotStrip public fun useFabricInterop(): Boolean
181181

182-
@DoNotStrip public fun useNativeTransformHelperAndroid(): Boolean
183-
184182
@DoNotStrip public fun useNativeViewConfigsInBridgelessMode(): Boolean
185183

186184
@DoNotStrip public fun useRawPropsJsiValue(): Boolean

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/TransformHelper.kt

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import com.facebook.react.bridge.ReadableArray
1313
import com.facebook.react.bridge.ReadableMap
1414
import com.facebook.react.bridge.ReadableType
1515
import com.facebook.react.common.ReactConstants
16-
import com.facebook.react.internal.featureflags.ReactNativeFeatureFlags
1716

1817
public object TransformHelper {
1918

@@ -73,12 +72,7 @@ public object TransformHelper {
7372
transformOrigin: ReadableArray?,
7473
allowPercentageResolution: Boolean,
7574
) {
76-
if (
77-
allowPercentageResolution &&
78-
ReactNativeFeatureFlags.useNativeTransformHelperAndroid() &&
79-
transforms is NativeArray &&
80-
transformOrigin is NativeArray?
81-
) {
75+
if (allowPercentageResolution && transforms is NativeArray && transformOrigin is NativeArray?) {
8276
nativeProcessTransform(transforms, result, viewWidth, viewHeight, transformOrigin)
8377
return
8478
}

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<<5a83d0e5570246e5d44a49d53b75b4fc>>
7+
* @generated SignedSource<<1ea556a16d18a54b7087742b986a93a7>>
88
*/
99

1010
/**
@@ -507,12 +507,6 @@ class ReactNativeFeatureFlagsJavaProvider
507507
return method(javaProvider_);
508508
}
509509

510-
bool useNativeTransformHelperAndroid() override {
511-
static const auto method =
512-
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("useNativeTransformHelperAndroid");
513-
return method(javaProvider_);
514-
}
515-
516510
bool useNativeViewConfigsInBridgelessMode() override {
517511
static const auto method =
518512
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("useNativeViewConfigsInBridgelessMode");
@@ -967,11 +961,6 @@ bool JReactNativeFeatureFlagsCxxInterop::useFabricInterop(
967961
return ReactNativeFeatureFlags::useFabricInterop();
968962
}
969963

970-
bool JReactNativeFeatureFlagsCxxInterop::useNativeTransformHelperAndroid(
971-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
972-
return ReactNativeFeatureFlags::useNativeTransformHelperAndroid();
973-
}
974-
975964
bool JReactNativeFeatureFlagsCxxInterop::useNativeViewConfigsInBridgelessMode(
976965
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
977966
return ReactNativeFeatureFlags::useNativeViewConfigsInBridgelessMode();
@@ -1287,9 +1276,6 @@ void JReactNativeFeatureFlagsCxxInterop::registerNatives() {
12871276
makeNativeMethod(
12881277
"useFabricInterop",
12891278
JReactNativeFeatureFlagsCxxInterop::useFabricInterop),
1290-
makeNativeMethod(
1291-
"useNativeTransformHelperAndroid",
1292-
JReactNativeFeatureFlagsCxxInterop::useNativeTransformHelperAndroid),
12931279
makeNativeMethod(
12941280
"useNativeViewConfigsInBridgelessMode",
12951281
JReactNativeFeatureFlagsCxxInterop::useNativeViewConfigsInBridgelessMode),

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<<fa32d0e44ae50a724f72f95c56d1418e>>
7+
* @generated SignedSource<<67b23d4209c5fa6b721921953d76a829>>
88
*/
99

1010
/**
@@ -264,9 +264,6 @@ class JReactNativeFeatureFlagsCxxInterop
264264
static bool useFabricInterop(
265265
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
266266

267-
static bool useNativeTransformHelperAndroid(
268-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
269-
270267
static bool useNativeViewConfigsInBridgelessMode(
271268
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
272269

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<<be750689f90d9498b3d44055c2346a0f>>
7+
* @generated SignedSource<<3d71ee2b906dcc1313428185cbe853f8>>
88
*/
99

1010
/**
@@ -338,10 +338,6 @@ bool ReactNativeFeatureFlags::useFabricInterop() {
338338
return getAccessor().useFabricInterop();
339339
}
340340

341-
bool ReactNativeFeatureFlags::useNativeTransformHelperAndroid() {
342-
return getAccessor().useNativeTransformHelperAndroid();
343-
}
344-
345341
bool ReactNativeFeatureFlags::useNativeViewConfigsInBridgelessMode() {
346342
return getAccessor().useNativeViewConfigsInBridgelessMode();
347343
}

0 commit comments

Comments
 (0)