Skip to content

Commit 02689c8

Browse files
Remove UIManagerCommitHookManager 1/2 (#54234)
Summary: Changelog: [Internal] This feature does not work reliably and caused crashes during experimentation Reviewed By: lenaic Differential Revision: D85259390
1 parent d93d325 commit 02689c8

24 files changed

+77
-302
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<<2ac9938108dfe555fc7e7d875cc21987>>
7+
* @generated SignedSource<<28000f9d4a36207875619060f2fa5713>>
88
*/
99

1010
/**
@@ -186,12 +186,6 @@ public object ReactNativeFeatureFlags {
186186
@JvmStatic
187187
public fun enableImagePrefetchingAndroid(): Boolean = accessor.enableImagePrefetchingAndroid()
188188

189-
/**
190-
* When enabled, Android will build and initiate image prefetch requests on ImageShadowNode::layout and batch them together in a single JNI call
191-
*/
192-
@JvmStatic
193-
public fun enableImagePrefetchingJNIBatchingAndroid(): Boolean = accessor.enableImagePrefetchingJNIBatchingAndroid()
194-
195189
/**
196190
* When enabled, Android will initiate image prefetch requested on ImageShadowNode::layout on the UI thread
197191
*/

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

1010
/**
@@ -46,7 +46,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
4646
private var enableIOSTextBaselineOffsetPerLineCache: Boolean? = null
4747
private var enableIOSViewClipToPaddingBoxCache: Boolean? = null
4848
private var enableImagePrefetchingAndroidCache: Boolean? = null
49-
private var enableImagePrefetchingJNIBatchingAndroidCache: Boolean? = null
5049
private var enableImagePrefetchingOnUiThreadAndroidCache: Boolean? = null
5150
private var enableImmediateUpdateModeForContentOffsetChangesCache: Boolean? = null
5251
private var enableImperativeFocusCache: Boolean? = null
@@ -336,15 +335,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
336335
return cached
337336
}
338337

339-
override fun enableImagePrefetchingJNIBatchingAndroid(): Boolean {
340-
var cached = enableImagePrefetchingJNIBatchingAndroidCache
341-
if (cached == null) {
342-
cached = ReactNativeFeatureFlagsCxxInterop.enableImagePrefetchingJNIBatchingAndroid()
343-
enableImagePrefetchingJNIBatchingAndroidCache = cached
344-
}
345-
return cached
346-
}
347-
348338
override fun enableImagePrefetchingOnUiThreadAndroid(): Boolean {
349339
var cached = enableImagePrefetchingOnUiThreadAndroidCache
350340
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<<de0fe8d116ca48e918f67c4c32f1183f>>
7+
* @generated SignedSource<<412a865e975214a0d794985e48fbbe4b>>
88
*/
99

1010
/**
@@ -80,8 +80,6 @@ public object ReactNativeFeatureFlagsCxxInterop {
8080

8181
@DoNotStrip @JvmStatic public external fun enableImagePrefetchingAndroid(): Boolean
8282

83-
@DoNotStrip @JvmStatic public external fun enableImagePrefetchingJNIBatchingAndroid(): Boolean
84-
8583
@DoNotStrip @JvmStatic public external fun enableImagePrefetchingOnUiThreadAndroid(): Boolean
8684

8785
@DoNotStrip @JvmStatic public external fun enableImmediateUpdateModeForContentOffsetChanges(): 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<<674f7910fe5dcd750ba2661122c82670>>
7+
* @generated SignedSource<<7987eba84c39c6e8cb6494092a94790e>>
88
*/
99

1010
/**
@@ -75,8 +75,6 @@ public open class ReactNativeFeatureFlagsDefaults : ReactNativeFeatureFlagsProvi
7575

7676
override fun enableImagePrefetchingAndroid(): Boolean = false
7777

78-
override fun enableImagePrefetchingJNIBatchingAndroid(): Boolean = false
79-
8078
override fun enableImagePrefetchingOnUiThreadAndroid(): Boolean = false
8179

8280
override fun enableImmediateUpdateModeForContentOffsetChanges(): Boolean = false

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<<5facf1328467d62b41dd9f82c5111882>>
7+
* @generated SignedSource<<7f43cde1ba2bef2300c077f35b168d31>>
88
*/
99

1010
/**
@@ -50,7 +50,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
5050
private var enableIOSTextBaselineOffsetPerLineCache: Boolean? = null
5151
private var enableIOSViewClipToPaddingBoxCache: Boolean? = null
5252
private var enableImagePrefetchingAndroidCache: Boolean? = null
53-
private var enableImagePrefetchingJNIBatchingAndroidCache: Boolean? = null
5453
private var enableImagePrefetchingOnUiThreadAndroidCache: Boolean? = null
5554
private var enableImmediateUpdateModeForContentOffsetChangesCache: Boolean? = null
5655
private var enableImperativeFocusCache: Boolean? = null
@@ -366,16 +365,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
366365
return cached
367366
}
368367

369-
override fun enableImagePrefetchingJNIBatchingAndroid(): Boolean {
370-
var cached = enableImagePrefetchingJNIBatchingAndroidCache
371-
if (cached == null) {
372-
cached = currentProvider.enableImagePrefetchingJNIBatchingAndroid()
373-
accessedFeatureFlags.add("enableImagePrefetchingJNIBatchingAndroid")
374-
enableImagePrefetchingJNIBatchingAndroidCache = cached
375-
}
376-
return cached
377-
}
378-
379368
override fun enableImagePrefetchingOnUiThreadAndroid(): Boolean {
380369
var cached = enableImagePrefetchingOnUiThreadAndroidCache
381370
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<<5a9a51638a72bda657f312260bb1297a>>
7+
* @generated SignedSource<<692dcdb2af3c6af981e8f48686ca105e>>
88
*/
99

1010
/**
@@ -75,8 +75,6 @@ public interface ReactNativeFeatureFlagsProvider {
7575

7676
@DoNotStrip public fun enableImagePrefetchingAndroid(): Boolean
7777

78-
@DoNotStrip public fun enableImagePrefetchingJNIBatchingAndroid(): Boolean
79-
8078
@DoNotStrip public fun enableImagePrefetchingOnUiThreadAndroid(): Boolean
8179

8280
@DoNotStrip public fun enableImmediateUpdateModeForContentOffsetChanges(): Boolean

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

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

1010
/**
@@ -195,12 +195,6 @@ class ReactNativeFeatureFlagsJavaProvider
195195
return method(javaProvider_);
196196
}
197197

198-
bool enableImagePrefetchingJNIBatchingAndroid() override {
199-
static const auto method =
200-
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableImagePrefetchingJNIBatchingAndroid");
201-
return method(javaProvider_);
202-
}
203-
204198
bool enableImagePrefetchingOnUiThreadAndroid() override {
205199
static const auto method =
206200
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableImagePrefetchingOnUiThreadAndroid");
@@ -659,11 +653,6 @@ bool JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingAndroid(
659653
return ReactNativeFeatureFlags::enableImagePrefetchingAndroid();
660654
}
661655

662-
bool JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingJNIBatchingAndroid(
663-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
664-
return ReactNativeFeatureFlags::enableImagePrefetchingJNIBatchingAndroid();
665-
}
666-
667656
bool JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingOnUiThreadAndroid(
668657
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
669658
return ReactNativeFeatureFlags::enableImagePrefetchingOnUiThreadAndroid();
@@ -1043,9 +1032,6 @@ void JReactNativeFeatureFlagsCxxInterop::registerNatives() {
10431032
makeNativeMethod(
10441033
"enableImagePrefetchingAndroid",
10451034
JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingAndroid),
1046-
makeNativeMethod(
1047-
"enableImagePrefetchingJNIBatchingAndroid",
1048-
JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingJNIBatchingAndroid),
10491035
makeNativeMethod(
10501036
"enableImagePrefetchingOnUiThreadAndroid",
10511037
JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingOnUiThreadAndroid),

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<<5789c040408e16ad3e5f79b169084c15>>
7+
* @generated SignedSource<<e8bbe1de8eccb3ecea6b9f001de07fd2>>
88
*/
99

1010
/**
@@ -108,9 +108,6 @@ class JReactNativeFeatureFlagsCxxInterop
108108
static bool enableImagePrefetchingAndroid(
109109
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
110110

111-
static bool enableImagePrefetchingJNIBatchingAndroid(
112-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
113-
114111
static bool enableImagePrefetchingOnUiThreadAndroid(
115112
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
116113

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<<d65469ea355ee606c40814c142a391f7>>
7+
* @generated SignedSource<<0554340092d31b5431e201f570599ab5>>
88
*/
99

1010
/**
@@ -130,10 +130,6 @@ bool ReactNativeFeatureFlags::enableImagePrefetchingAndroid() {
130130
return getAccessor().enableImagePrefetchingAndroid();
131131
}
132132

133-
bool ReactNativeFeatureFlags::enableImagePrefetchingJNIBatchingAndroid() {
134-
return getAccessor().enableImagePrefetchingJNIBatchingAndroid();
135-
}
136-
137133
bool ReactNativeFeatureFlags::enableImagePrefetchingOnUiThreadAndroid() {
138134
return getAccessor().enableImagePrefetchingOnUiThreadAndroid();
139135
}

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

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

1010
/**
@@ -169,11 +169,6 @@ class ReactNativeFeatureFlags {
169169
*/
170170
RN_EXPORT static bool enableImagePrefetchingAndroid();
171171

172-
/**
173-
* When enabled, Android will build and initiate image prefetch requests on ImageShadowNode::layout and batch them together in a single JNI call
174-
*/
175-
RN_EXPORT static bool enableImagePrefetchingJNIBatchingAndroid();
176-
177172
/**
178173
* When enabled, Android will initiate image prefetch requested on ImageShadowNode::layout on the UI thread
179174
*/

0 commit comments

Comments
 (0)