Skip to content

Commit ad8be9a

Browse files
Abbondanzofacebook-github-bot
authored andcommitted
Remove Android image prefetching feature flag (#50412)
Summary: Pull Request resolved: #50412 Removes the `enableImagePrefetchingAndroid` feature flag. Changelog: [Internal] Reviewed By: lenaic Differential Revision: D72198491 fbshipit-source-id: bd9bb9ecc7b58ac1fe6a507ec9caf1ce99e0ba63
1 parent 4912958 commit ad8be9a

24 files changed

Lines changed: 54 additions & 181 deletions

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

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

1010
/**
@@ -96,12 +96,6 @@ public object ReactNativeFeatureFlags {
9696
@JvmStatic
9797
public fun enableIOSViewClipToPaddingBox(): Boolean = accessor.enableIOSViewClipToPaddingBox()
9898

99-
/**
100-
* When enabled, Android will build and initiate image prefetch requests on ImageShadowNode::layout
101-
*/
102-
@JvmStatic
103-
public fun enableImagePrefetchingAndroid(): Boolean = accessor.enableImagePrefetchingAndroid()
104-
10599
/**
106100
* Trigger JS runtime GC on memory pressure event on iOS
107101
*/

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<<7196636e08059866a9324ab57609e492>>
7+
* @generated SignedSource<<23e555f1c49609681b879815e5f53da7>>
88
*/
99

1010
/**
@@ -31,7 +31,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
3131
private var enableFabricLogsCache: Boolean? = null
3232
private var enableFabricRendererCache: Boolean? = null
3333
private var enableIOSViewClipToPaddingBoxCache: Boolean? = null
34-
private var enableImagePrefetchingAndroidCache: Boolean? = null
3534
private var enableJSRuntimeGCOnMemoryPressureOnIOSCache: Boolean? = null
3635
private var enableLayoutAnimationsOnAndroidCache: Boolean? = null
3736
private var enableLayoutAnimationsOnIOSCache: Boolean? = null
@@ -162,15 +161,6 @@ internal class ReactNativeFeatureFlagsCxxAccessor : ReactNativeFeatureFlagsAcces
162161
return cached
163162
}
164163

165-
override fun enableImagePrefetchingAndroid(): Boolean {
166-
var cached = enableImagePrefetchingAndroidCache
167-
if (cached == null) {
168-
cached = ReactNativeFeatureFlagsCxxInterop.enableImagePrefetchingAndroid()
169-
enableImagePrefetchingAndroidCache = cached
170-
}
171-
return cached
172-
}
173-
174164
override fun enableJSRuntimeGCOnMemoryPressureOnIOS(): Boolean {
175165
var cached = enableJSRuntimeGCOnMemoryPressureOnIOSCache
176166
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<<463515a955baca809dd6463107ca325c>>
7+
* @generated SignedSource<<8989c14be4a2ef16890f0dc06ea8bcc2>>
88
*/
99

1010
/**
@@ -50,8 +50,6 @@ public object ReactNativeFeatureFlagsCxxInterop {
5050

5151
@DoNotStrip @JvmStatic public external fun enableIOSViewClipToPaddingBox(): Boolean
5252

53-
@DoNotStrip @JvmStatic public external fun enableImagePrefetchingAndroid(): Boolean
54-
5553
@DoNotStrip @JvmStatic public external fun enableJSRuntimeGCOnMemoryPressureOnIOS(): Boolean
5654

5755
@DoNotStrip @JvmStatic public external fun enableLayoutAnimationsOnAndroid(): 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<<309ae9cf008963de368916b3d28c5491>>
7+
* @generated SignedSource<<753948273e5701090a7cfb7d1582eb00>>
88
*/
99

1010
/**
@@ -45,8 +45,6 @@ public open class ReactNativeFeatureFlagsDefaults : ReactNativeFeatureFlagsProvi
4545

4646
override fun enableIOSViewClipToPaddingBox(): Boolean = false
4747

48-
override fun enableImagePrefetchingAndroid(): Boolean = false
49-
5048
override fun enableJSRuntimeGCOnMemoryPressureOnIOS(): Boolean = false
5149

5250
override fun enableLayoutAnimationsOnAndroid(): 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<<6171c6bc5492b1a0252746e928754f97>>
7+
* @generated SignedSource<<543d7b7cca5d15dc8205af831c304b57>>
88
*/
99

1010
/**
@@ -35,7 +35,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
3535
private var enableFabricLogsCache: Boolean? = null
3636
private var enableFabricRendererCache: Boolean? = null
3737
private var enableIOSViewClipToPaddingBoxCache: Boolean? = null
38-
private var enableImagePrefetchingAndroidCache: Boolean? = null
3938
private var enableJSRuntimeGCOnMemoryPressureOnIOSCache: Boolean? = null
4039
private var enableLayoutAnimationsOnAndroidCache: Boolean? = null
4140
private var enableLayoutAnimationsOnIOSCache: Boolean? = null
@@ -177,16 +176,6 @@ internal class ReactNativeFeatureFlagsLocalAccessor : ReactNativeFeatureFlagsAcc
177176
return cached
178177
}
179178

180-
override fun enableImagePrefetchingAndroid(): Boolean {
181-
var cached = enableImagePrefetchingAndroidCache
182-
if (cached == null) {
183-
cached = currentProvider.enableImagePrefetchingAndroid()
184-
accessedFeatureFlags.add("enableImagePrefetchingAndroid")
185-
enableImagePrefetchingAndroidCache = cached
186-
}
187-
return cached
188-
}
189-
190179
override fun enableJSRuntimeGCOnMemoryPressureOnIOS(): Boolean {
191180
var cached = enableJSRuntimeGCOnMemoryPressureOnIOSCache
192181
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<<a4f3222340cd676eefdd3794d3f1d4da>>
7+
* @generated SignedSource<<56605e18396622084f170b5a897fa0e0>>
88
*/
99

1010
/**
@@ -45,8 +45,6 @@ public interface ReactNativeFeatureFlagsProvider {
4545

4646
@DoNotStrip public fun enableIOSViewClipToPaddingBox(): Boolean
4747

48-
@DoNotStrip public fun enableImagePrefetchingAndroid(): Boolean
49-
5048
@DoNotStrip public fun enableJSRuntimeGCOnMemoryPressureOnIOS(): Boolean
5149

5250
@DoNotStrip public fun enableLayoutAnimationsOnAndroid(): Boolean

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -540,6 +540,6 @@ public void experimental_prefetchResource(
540540

541541
@UnstableReactNativeAPI
542542
protected boolean experimental_isPrefetchingEnabled() {
543-
return ReactNativeFeatureFlags.enableImagePrefetchingAndroid();
543+
return false;
544544
}
545545
}

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<<2e1dacd320fd57c091996e3e8e2843a9>>
7+
* @generated SignedSource<<d9aeb8e0dd7091a1ee2d1510c5544a9f>>
88
*/
99

1010
/**
@@ -105,12 +105,6 @@ class ReactNativeFeatureFlagsJavaProvider
105105
return method(javaProvider_);
106106
}
107107

108-
bool enableImagePrefetchingAndroid() override {
109-
static const auto method =
110-
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableImagePrefetchingAndroid");
111-
return method(javaProvider_);
112-
}
113-
114108
bool enableJSRuntimeGCOnMemoryPressureOnIOS() override {
115109
static const auto method =
116110
getReactNativeFeatureFlagsProviderJavaClass()->getMethod<jboolean()>("enableJSRuntimeGCOnMemoryPressureOnIOS");
@@ -350,11 +344,6 @@ bool JReactNativeFeatureFlagsCxxInterop::enableIOSViewClipToPaddingBox(
350344
return ReactNativeFeatureFlags::enableIOSViewClipToPaddingBox();
351345
}
352346

353-
bool JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingAndroid(
354-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
355-
return ReactNativeFeatureFlags::enableImagePrefetchingAndroid();
356-
}
357-
358347
bool JReactNativeFeatureFlagsCxxInterop::enableJSRuntimeGCOnMemoryPressureOnIOS(
359348
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop> /*unused*/) {
360349
return ReactNativeFeatureFlags::enableJSRuntimeGCOnMemoryPressureOnIOS();
@@ -569,9 +558,6 @@ void JReactNativeFeatureFlagsCxxInterop::registerNatives() {
569558
makeNativeMethod(
570559
"enableIOSViewClipToPaddingBox",
571560
JReactNativeFeatureFlagsCxxInterop::enableIOSViewClipToPaddingBox),
572-
makeNativeMethod(
573-
"enableImagePrefetchingAndroid",
574-
JReactNativeFeatureFlagsCxxInterop::enableImagePrefetchingAndroid),
575561
makeNativeMethod(
576562
"enableJSRuntimeGCOnMemoryPressureOnIOS",
577563
JReactNativeFeatureFlagsCxxInterop::enableJSRuntimeGCOnMemoryPressureOnIOS),

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<<d0b8fdd82018fab8e5532123d5254cf4>>
7+
* @generated SignedSource<<a67867e3cad96e89607dc388d8612eef>>
88
*/
99

1010
/**
@@ -63,9 +63,6 @@ class JReactNativeFeatureFlagsCxxInterop
6363
static bool enableIOSViewClipToPaddingBox(
6464
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
6565

66-
static bool enableImagePrefetchingAndroid(
67-
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
68-
6966
static bool enableJSRuntimeGCOnMemoryPressureOnIOS(
7067
facebook::jni::alias_ref<JReactNativeFeatureFlagsCxxInterop>);
7168

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<<ae8953b9c5b0efae6fe67b9b799426d7>>
7+
* @generated SignedSource<<2ba5830499c05aad7e1bb58668027a41>>
88
*/
99

1010
/**
@@ -70,10 +70,6 @@ bool ReactNativeFeatureFlags::enableIOSViewClipToPaddingBox() {
7070
return getAccessor().enableIOSViewClipToPaddingBox();
7171
}
7272

73-
bool ReactNativeFeatureFlags::enableImagePrefetchingAndroid() {
74-
return getAccessor().enableImagePrefetchingAndroid();
75-
}
76-
7773
bool ReactNativeFeatureFlags::enableJSRuntimeGCOnMemoryPressureOnIOS() {
7874
return getAccessor().enableJSRuntimeGCOnMemoryPressureOnIOS();
7975
}

0 commit comments

Comments
 (0)