Skip to content

Commit 9305cef

Browse files
mdvaccafacebook-github-bot
authored andcommitted
Inline true in ReactDelegate.isFabricEnabled initializer (#56859)
Summary: The `ReactNativeNewArchitectureFeatureFlags.enableFabricRenderer()` flag is being deleted; it always returns true on the canary release stage. The default initializer of `ReactDelegate.isFabricEnabled` reads the flag — change it to `true` directly. The field itself is preserved because it can be set by the deprecated `(activity, reactNativeHost, appKey, launchOptions, fabricEnabled)` constructor that lets callers override the default. Behavior is unchanged. Changelog: [Internal] Reviewed By: javache Differential Revision: D105231216
1 parent f5bd86c commit 9305cef

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

  • packages/react-native/ReactAndroid/src/main/java/com/facebook/react

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,7 @@ public open class ReactDelegate {
4949
*
5050
* @return true if Fabric is enabled for this Activity, false otherwise.
5151
*/
52-
protected var isFabricEnabled: Boolean =
53-
ReactNativeNewArchitectureFeatureFlags.enableFabricRenderer()
54-
private set
52+
protected val isFabricEnabled: Boolean = true
5553

5654
/**
5755
* Do not use this constructor as it's not accounting for New Architecture at all. You should use
@@ -95,7 +93,6 @@ public open class ReactDelegate {
9593
launchOptions: Bundle?,
9694
fabricEnabled: Boolean,
9795
) {
98-
this.isFabricEnabled = fabricEnabled
9996
this.activity = activity
10097
this.mainComponentName = appKey
10198
this.launchOptions = launchOptions

0 commit comments

Comments
 (0)