Skip to content

Commit 935f8e5

Browse files
javachemeta-codesync[bot]
authored andcommitted
Deprecate legacy DefaultReactActivityDelegate constructors (#56906)
Summary: Pull Request resolved: #56906 These flags were already ignored. Changelog: [Android][Deprecated] DefaultReactActivityDelegate's constructor taking new arch flags are deprecated Reviewed By: cortinico Differential Revision: D105816717 fbshipit-source-id: 193ca28658b024f93014334ab2622d64427f31a5
1 parent 9cbddbd commit 935f8e5

3 files changed

Lines changed: 16 additions & 14 deletions

File tree

packages/react-native/ReactAndroid/api/ReactAndroid.api

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1750,10 +1750,9 @@ public final class com/facebook/react/defaults/DefaultNewArchitectureEntryPoint
17501750
}
17511751

17521752
public class com/facebook/react/defaults/DefaultReactActivityDelegate : com/facebook/react/ReactActivityDelegate {
1753+
public fun <init> (Lcom/facebook/react/ReactActivity;Ljava/lang/String;)V
17531754
public fun <init> (Lcom/facebook/react/ReactActivity;Ljava/lang/String;Z)V
1754-
public synthetic fun <init> (Lcom/facebook/react/ReactActivity;Ljava/lang/String;ZILkotlin/jvm/internal/DefaultConstructorMarker;)V
17551755
public fun <init> (Lcom/facebook/react/ReactActivity;Ljava/lang/String;ZZ)V
1756-
protected fun isFabricEnabled ()Z
17571756
}
17581757

17591758
public final class com/facebook/react/defaults/DefaultReactHost {

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

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -22,24 +22,28 @@ import com.facebook.react.ReactActivityDelegate
2222
public open class DefaultReactActivityDelegate(
2323
activity: ReactActivity,
2424
mainComponentName: String,
25-
private val fabricEnabled: Boolean = false,
2625
) : ReactActivityDelegate(activity, mainComponentName) {
2726

2827
@Deprecated(
29-
message =
30-
"Creating DefaultReactActivityDelegate with both fabricEnabled and " +
31-
"concurrentReactEnabled is deprecated. Please pass only one boolean value that will" +
32-
" be used for both flags",
28+
message = "Creating DefaultReactActivityDelegate with flags is deprecated.",
3329
level = DeprecationLevel.WARNING,
34-
replaceWith =
35-
ReplaceWith("DefaultReactActivityDelegate(activity, mainComponentName, fabricEnabled)"),
30+
replaceWith = ReplaceWith("DefaultReactActivityDelegate(activity, mainComponentName)"),
3631
)
3732
public constructor(
3833
activity: ReactActivity,
3934
mainComponentName: String,
40-
fabricEnabled: Boolean,
35+
@Suppress("UNUSED_PARAMETER") fabricEnabled: Boolean,
4136
@Suppress("UNUSED_PARAMETER") concurrentReactEnabled: Boolean,
42-
) : this(activity, mainComponentName, fabricEnabled)
37+
) : this(activity, mainComponentName)
4338

44-
override fun isFabricEnabled(): Boolean = fabricEnabled
39+
@Deprecated(
40+
message = "Creating DefaultReactActivityDelegate with flags is deprecated.",
41+
level = DeprecationLevel.WARNING,
42+
replaceWith = ReplaceWith("DefaultReactActivityDelegate(activity, mainComponentName)"),
43+
)
44+
public constructor(
45+
activity: ReactActivity,
46+
mainComponentName: String,
47+
@Suppress("UNUSED_PARAMETER") fabricEnabled: Boolean,
48+
) : this(activity, mainComponentName)
4549
}

packages/rn-tester/android/app/src/main/java/com/facebook/react/uiapp/RNTesterActivity.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,14 @@ import androidx.core.view.ViewCompat
1717
import androidx.core.view.WindowInsetsCompat
1818
import com.facebook.react.FBRNTesterEndToEndHelper
1919
import com.facebook.react.ReactActivity
20-
import com.facebook.react.defaults.DefaultNewArchitectureEntryPoint.fabricEnabled
2120
import com.facebook.react.defaults.DefaultReactActivityDelegate
2221
import com.facebook.react.devsupport.DevMenuConfiguration
2322
import java.io.FileDescriptor
2423
import java.io.PrintWriter
2524

2625
internal class RNTesterActivity : ReactActivity() {
2726
class RNTesterActivityDelegate(val activity: ReactActivity, mainComponentName: String) :
28-
DefaultReactActivityDelegate(activity, mainComponentName, fabricEnabled) {
27+
DefaultReactActivityDelegate(activity, mainComponentName) {
2928
private val PARAM_ROUTE = "route"
3029
private lateinit var initialProps: Bundle
3130

0 commit comments

Comments
 (0)