Skip to content

Commit 8254758

Browse files
mdvaccafacebook-github-bot
authored andcommitted
Annotate Legacy Architecture classes of com/facebook/react (#49765)
Summary: Pull Request resolved: #49765 Annotate Legacy Architecture classes of com/facebook/react changelog: [internal] internal Reviewed By: rshest Differential Revision: D70441494 fbshipit-source-id: bc677ce508a0d119258819fc6a02d339dc5d8e9a
1 parent 7d71f5d commit 8254758

8 files changed

Lines changed: 16 additions & 1 deletion

File tree

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
import com.facebook.react.bridge.ReactApplicationContext;
1818
import com.facebook.react.bridge.ReactMarker;
1919
import com.facebook.react.common.ClassFinder;
20+
import com.facebook.react.common.annotations.internal.LegacyArchitecture;
2021
import com.facebook.react.devsupport.LogBoxModule;
2122
import com.facebook.react.module.annotations.ReactModule;
2223
import com.facebook.react.module.annotations.ReactModuleList;
@@ -59,6 +60,7 @@
5960
TimingModule.class,
6061
UIManagerModule.class,
6162
})
63+
@LegacyArchitecture
6264
class CoreModulesPackage extends BaseReactPackage implements ReactPackageLogger {
6365

6466
private final ReactInstanceManager mReactInstanceManager;

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,15 @@
66
*/
77

88
package com.facebook.react
9+
10+
import com.facebook.react.common.annotations.internal.LegacyArchitecture
11+
912
/**
1013
* An enum that specifies the algorithm to use when loading theJS Engine. [.JSC] will load
1114
* JavaScriptCore first and fail if it is not available. [.HERMES] will load Hermes first and fail
1215
* if it is not available.
1316
*/
17+
@LegacyArchitecture
1418
public enum class JSEngineResolutionAlgorithm {
1519
JSC,
1620
HERMES

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import com.facebook.react.bridge.ReactApplicationContext;
1616
import com.facebook.react.bridge.ReactMarker;
1717
import com.facebook.react.bridge.ReactMarkerConstants;
18+
import com.facebook.react.common.annotations.internal.LegacyArchitecture;
1819
import com.facebook.react.module.model.ReactModuleInfo;
1920
import com.facebook.react.module.model.ReactModuleInfoProvider;
2021
import com.facebook.react.uimanager.ViewManager;
@@ -27,6 +28,7 @@
2728

2829
/** React package supporting lazy creation of native modules. */
2930
@Deprecated(since = "This class is deprecated, please use BaseReactPackage instead.")
31+
@LegacyArchitecture
3032
public abstract class LazyReactPackage implements ReactPackage {
3133

3234
/**

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,10 @@ package com.facebook.react
1010
import com.facebook.react.bridge.ModuleHolder
1111
import com.facebook.react.bridge.NativeModuleRegistry
1212
import com.facebook.react.bridge.ReactApplicationContext
13+
import com.facebook.react.common.annotations.internal.LegacyArchitecture
1314

1415
/** Helper class to build NativeModuleRegistry. */
16+
@LegacyArchitecture
1517
public class NativeModuleRegistryBuilder(
1618
private val reactApplicationContext: ReactApplicationContext,
1719
) {

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
import com.facebook.react.common.SurfaceDelegateFactory;
7979
import com.facebook.react.common.annotations.StableReactNativeAPI;
8080
import com.facebook.react.common.annotations.VisibleForTesting;
81+
import com.facebook.react.common.annotations.internal.LegacyArchitecture;
8182
import com.facebook.react.devsupport.DevSupportManagerFactory;
8283
import com.facebook.react.devsupport.InspectorFlags;
8384
import com.facebook.react.devsupport.ReactInstanceDevHelper;
@@ -143,6 +144,7 @@
143144
*/
144145
@ThreadSafe
145146
@StableReactNativeAPI
147+
@LegacyArchitecture
146148
public class ReactInstanceManager {
147149

148150
private static final String TAG = ReactInstanceManager.class.getSimpleName();

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import com.facebook.react.common.LifecycleState;
2626
import com.facebook.react.common.SurfaceDelegateFactory;
2727
import com.facebook.react.common.annotations.StableReactNativeAPI;
28+
import com.facebook.react.common.annotations.internal.LegacyArchitecture;
2829
import com.facebook.react.devsupport.DefaultDevSupportManagerFactory;
2930
import com.facebook.react.devsupport.DevSupportManagerFactory;
3031
import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
@@ -44,6 +45,7 @@
4445

4546
/** Builder class for {@link ReactInstanceManager} */
4647
@StableReactNativeAPI
48+
@LegacyArchitecture
4749
public class ReactInstanceManagerBuilder {
4850

4951
private static final String TAG = ReactInstanceManagerBuilder.class.getSimpleName();

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

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
import com.facebook.react.common.SurfaceDelegate;
2020
import com.facebook.react.common.SurfaceDelegateFactory;
2121
import com.facebook.react.common.annotations.DeprecatedInNewArchitecture;
22+
import com.facebook.react.common.annotations.internal.LegacyArchitecture;
2223
import com.facebook.react.devsupport.DevSupportManagerFactory;
2324
import com.facebook.react.devsupport.interfaces.DevLoadingViewManager;
2425
import com.facebook.react.devsupport.interfaces.PausedInDebuggerOverlayManager;
@@ -34,6 +35,7 @@
3435
message =
3536
"This class will be replaced by com.facebook.react.ReactHost in the new architecture of"
3637
+ " React Native.")
38+
@LegacyArchitecture
3739
public abstract class ReactNativeHost {
3840

3941
private final Application mApplication;

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ internal object ReactPackageHelper {
1919
*
2020
* @param reactPackage
2121
* @param reactApplicationContext
22-
* @param reactInstanceManager
2322
* @return
2423
*/
2524
fun getNativeModuleIterator(

0 commit comments

Comments
 (0)