Do not guard initializeFlipper for bridgeless for RN Tester

cortinico · cortinico · commit f79ec4bebb4c · 2023-10-18T14:16:29.000+01:00
Summary:
We don't need this if-than-else because the initializeFlipper already checks if we're on bridgeless or not

Changelog:
[Internal] [Changed] - Do not guard initializeFlipper for bridgeless for RN Tester

Reviewed By: NickGerleman

Differential Revision: D49881903

fbshipit-source-id: e6bfc941b43382580bd418a5f27ad9426d300c69
diff --git a/packages/rn-tester/android/app/src/main/java/com/facebook/react/uiapp/RNTesterApplication.kt b/packages/rn-tester/android/app/src/main/java/com/facebook/react/uiapp/RNTesterApplication.kt
@@ -124,11 +124,7 @@ class RNTesterApplication : Application(), ReactApplication {
     if (BuildConfig.IS_NEW_ARCHITECTURE_ENABLED) {
       load()
     }
-    if (ReactFeatureFlags.enableBridgelessArchitecture) {
-      // TODO: initialize Flipper for Bridgeless
-    } else {
-      initializeFlipper(this, reactNativeHost.reactInstanceManager)
-    }
+    initializeFlipper(this, reactNativeHost.reactInstanceManager)
   }
 
   @UnstableReactNativeAPI

Original file line number	Diff line number	Diff line change
`@@ -124,11 +124,7 @@ class RNTesterApplication : Application(), ReactApplication {`
`124`	`124`	`if (BuildConfig.IS_NEW_ARCHITECTURE_ENABLED) {`
`125`	`125`	`load()`
`126`	`126`	`}`
`127`		`- if (ReactFeatureFlags.enableBridgelessArchitecture) {`
`128`		`- // TODO: initialize Flipper for Bridgeless`
`129`		`- } else {`
`130`		`- initializeFlipper(this, reactNativeHost.reactInstanceManager)`
`131`		`- }`
	`127`	`+ initializeFlipper(this, reactNativeHost.reactInstanceManager)`
`132`	`128`	`}`
`133`	`129`
`134`	`130`	`@UnstableReactNativeAPI`