Log class loading for subset of classes annotated with @LegacyArchitecture (#49831)

Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/49831

Log class loading for classes annotated with LegacyArchitecture
changelog: [internal] internal

Reviewed By: cortinico

Differential Revision: D70608476

fbshipit-source-id: 1efd37ec824c9d48b2e85439b23ca2caf62d0b20

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

@@ -18,6 +18,8 @@ import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReactMarker;
 import com.facebook.react.common.ClassFinder;
 import com.facebook.react.common.annotations.internal.LegacyArchitecture;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger;
 import com.facebook.react.devsupport.LogBoxModule;
 import com.facebook.react.module.annotations.ReactModule;
 import com.facebook.react.module.annotations.ReactModuleList;
@@ -63,6 +65,11 @@ import java.util.Map;
 @LegacyArchitecture
 class CoreModulesPackage extends BaseReactPackage implements ReactPackageLogger {
 
+  static {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "CoreModulesPackage", LegacyArchitectureLogLevel.WARNING);
+  }
+
   private final ReactInstanceManager mReactInstanceManager;
   private final DefaultHardwareBackBtnHandler mHardwareBackBtnHandler;
   private final boolean mLazyViewManagersEnabled;

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

@@ -17,6 +17,8 @@ import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReactMarker;
 import com.facebook.react.bridge.ReactMarkerConstants;
 import com.facebook.react.common.annotations.internal.LegacyArchitecture;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger;
 import com.facebook.react.module.model.ReactModuleInfo;
 import com.facebook.react.module.model.ReactModuleInfoProvider;
 import com.facebook.react.uimanager.ViewManager;
@@ -33,6 +35,11 @@ import java.util.Map;
 @LegacyArchitecture
 public abstract class LazyReactPackage implements ReactPackage {
 
+  static {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "LazyReactPackage", LegacyArchitectureLogLevel.WARNING);
+  }
+
   /**
    * We return an iterable
    *

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

@@ -26,6 +26,8 @@ import com.facebook.react.common.LifecycleState;
 import com.facebook.react.common.SurfaceDelegateFactory;
 import com.facebook.react.common.annotations.StableReactNativeAPI;
 import com.facebook.react.common.annotations.internal.LegacyArchitecture;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger;
 import com.facebook.react.devsupport.DefaultDevSupportManagerFactory;
 import com.facebook.react.devsupport.DevSupportManagerFactory;
 import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
@@ -48,6 +50,11 @@ import java.util.Map;
 @LegacyArchitecture
 public class ReactInstanceManagerBuilder {
 
+  static {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "ReactInstanceManagerBuilder", LegacyArchitectureLogLevel.WARNING);
+  }
+
   private static final String TAG = ReactInstanceManagerBuilder.class.getSimpleName();
 
   private final List<ReactPackage> mPackages = new ArrayList<>();

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

@@ -20,6 +20,8 @@ import com.facebook.react.common.SurfaceDelegate;
 import com.facebook.react.common.SurfaceDelegateFactory;
 import com.facebook.react.common.annotations.DeprecatedInNewArchitecture;
 import com.facebook.react.common.annotations.internal.LegacyArchitecture;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger;
 import com.facebook.react.devsupport.DevSupportManagerFactory;
 import com.facebook.react.devsupport.interfaces.DevLoadingViewManager;
 import com.facebook.react.devsupport.interfaces.PausedInDebuggerOverlayManager;
@@ -38,6 +40,11 @@ import java.util.List;
 @LegacyArchitecture
 public abstract class ReactNativeHost {
 
+  static {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "ReactNativeHost", LegacyArchitectureLogLevel.WARNING);
+  }
+
   private final Application mApplication;
   private @Nullable ReactInstanceManager mReactInstanceManager;
 

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

@@ -19,6 +19,8 @@ import com.facebook.react.bridge.ReactMarkerConstants;
 import com.facebook.react.bridge.UiThreadUtil;
 import com.facebook.react.common.SurfaceDelegateFactory;
 import com.facebook.react.common.annotations.internal.LegacyArchitecture;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogLevel;
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger;
 import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener;
 import com.facebook.react.devsupport.interfaces.DevLoadingViewManager;
 import com.facebook.react.devsupport.interfaces.DevSplitBundleCallback;
@@ -50,6 +52,12 @@ import java.util.Map;
  */
 @LegacyArchitecture
 public final class BridgeDevSupportManager extends DevSupportManagerBase {
+
+  static {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "BridgeDevSupportManager", LegacyArchitectureLogLevel.WARNING);
+  }
+
   private boolean mIsSamplingProfilerEnabled = false;
 
   public BridgeDevSupportManager(

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/interop/InteropEvent.kt

@@ -9,6 +9,7 @@ package com.facebook.react.internal.interop
 
 import com.facebook.react.bridge.WritableMap
 import com.facebook.react.common.annotations.internal.LegacyArchitecture
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 import com.facebook.react.uimanager.events.Event
 
 /**
@@ -23,6 +24,10 @@ public class InteropEvent(
     surfaceId: Int,
     viewTag: Int
 ) : Event<InteropEvent>(surfaceId, viewTag) {
+  init {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled("InteropEvent")
+  }
+
   override fun getEventName(): String = eventName
 
   override fun getEventData(): WritableMap? = eventData

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/internal/interop/InteropEventEmitter.kt

@@ -14,6 +14,7 @@ import com.facebook.react.bridge.WritableArray
 import com.facebook.react.bridge.WritableMap
 import com.facebook.react.common.annotations.VisibleForTesting
 import com.facebook.react.common.annotations.internal.LegacyArchitecture
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 import com.facebook.react.uimanager.UIManagerHelper
 import com.facebook.react.uimanager.events.EventDispatcher
 import com.facebook.react.uimanager.events.RCTEventEmitter
@@ -30,6 +31,10 @@ import com.facebook.react.uimanager.events.RCTEventEmitter
  */
 @LegacyArchitecture
 public class InteropEventEmitter(private val reactContext: ReactContext) : RCTEventEmitter {
+  init {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled("InteropEventEmitter")
+  }
+
   private var eventDispatcherOverride: EventDispatcher? = null
 
   @Deprecated("Deprecated in Java")

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

@@ -12,12 +12,17 @@ import com.facebook.proguard.annotations.DoNotStrip
 import com.facebook.react.bridge.JavaScriptExecutor
 import com.facebook.react.bridge.ReadableNativeMap
 import com.facebook.react.common.annotations.internal.LegacyArchitecture
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 import com.facebook.soloader.SoLoader
 
 @DoNotStrip
 @LegacyArchitecture
 public class JSCExecutor internal constructor(jscConfig: ReadableNativeMap) :
     JavaScriptExecutor(initHybrid(jscConfig)) {
+  init {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled("JSCExecutor")
+  }
+
   override fun getName(): String {
     return "JSCExecutor"
   }

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

@@ -11,11 +11,16 @@ import com.facebook.react.bridge.JavaScriptExecutor
 import com.facebook.react.bridge.JavaScriptExecutorFactory
 import com.facebook.react.bridge.WritableNativeMap
 import com.facebook.react.common.annotations.internal.LegacyArchitecture
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 
 @LegacyArchitecture
 public class JSCExecutorFactory(private val appName: String, private val deviceName: String) :
     JavaScriptExecutorFactory {
 
+  init {
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled("JSCExecutorFactory")
+  }
+
   @Throws(Exception::class)
   override fun create(): JavaScriptExecutor {
     val jscConfig =

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/layoutanimation/PositionAndSizeAnimation.kt

@@ -11,6 +11,7 @@ import android.view.View
 import android.view.animation.Animation
 import android.view.animation.Transformation
 import com.facebook.react.common.annotations.internal.LegacyArchitecture
+import com.facebook.react.common.annotations.internal.LegacyArchitectureLogger
 
 /**
  * Animation responsible for updating size and position of a view. We can't use scaling as view
@@ -37,6 +38,8 @@ internal class PositionAndSizeAnimation(
 
   init {
     calculateAnimation(x, y, width, height)
+    LegacyArchitectureLogger.assertWhenLegacyArchitectureMinifyingEnabled(
+        "PositionAndSizeAnimation")
   }
 
   override fun applyTransformation(interpolatedTime: Float, t: Transformation) {