Mark classes of package fabric as @nullsafe (#42725)

Summary:
Pull Request resolved: #42725

All these classes are NullSafe, let's mark them as NullSafe(Local) to ensure lint detect errors in the future

changelog: [internal] internal

Reviewed By: NickGerleman

Differential Revision: D53200097

fbshipit-source-id: dad54f5bf03967b5d4126757ab0d5424534af888

Author: mdvacca

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

@@ -9,6 +9,7 @@
 
 import android.annotation.SuppressLint;
 import androidx.annotation.NonNull;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 import com.facebook.react.bridge.NativeMap;
@@ -20,6 +21,7 @@
 import com.facebook.react.fabric.events.EventEmitterWrapper;
 import com.facebook.react.uimanager.PixelUtil;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 @DoNotStrip
 @SuppressLint("MissingNativeLoadLibrary")
 public class BindingImpl implements Binding {

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

@@ -8,9 +8,11 @@
 package com.facebook.react.fabric;
 
 import androidx.annotation.NonNull;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 @DoNotStrip
 public class ComponentFactory {
 

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

@@ -7,9 +7,11 @@
 
 package com.facebook.react.fabric;
 
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 @DoNotStrip
 public class CoreComponentsRegistry {
   static {

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

@@ -22,13 +22,15 @@
 import static com.facebook.react.bridge.ReactMarkerConstants.FABRIC_UPDATE_UI_MAIN_THREAD_START;
 
 import androidx.annotation.Nullable;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.react.bridge.ReactMarker;
 import com.facebook.react.bridge.ReactMarkerConstants;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 public class DevToolsReactPerfLogger implements ReactMarker.FabricMarkerListener {
 
   private final Map<Integer, FabricCommitPoint> mFabricCommitMarkers = new HashMap<>();

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

@@ -8,6 +8,7 @@
 package com.facebook.react.fabric;
 
 import androidx.annotation.NonNull;
+import com.facebook.infer.annotation.Nullsafe;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -17,6 +18,7 @@
  * <p>TODO T97384889: remove this class when the component names are unified between JS - Android -
  * iOS - C++
  */
+@Nullsafe(Nullsafe.Mode.LOCAL)
 public class FabricComponents {
 
   private static @NonNull final Map<String, String> sComponentNames = new HashMap<>();

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

@@ -9,11 +9,13 @@
 
 import static com.facebook.systrace.Systrace.TRACE_TAG_REACT_JAVA_BRIDGE;
 
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.react.bridge.ReactMarker;
 import com.facebook.react.bridge.ReactMarkerConstants;
 import com.facebook.soloader.SoLoader;
 import com.facebook.systrace.Systrace;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 public class FabricSoLoader {
   private static volatile boolean sDidInit = false;
 

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

@@ -9,9 +9,11 @@
 
 import android.view.Choreographer;
 import androidx.annotation.NonNull;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.react.bridge.JSExceptionHandler;
 import com.facebook.react.bridge.ReactContext;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 public abstract class GuardedFrameCallback implements Choreographer.FrameCallback {
 
   @NonNull private final JSExceptionHandler mExceptionHandler;

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

@@ -11,6 +11,7 @@
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import com.facebook.common.logging.FLog;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 import com.facebook.react.bridge.NativeMap;
@@ -23,6 +24,7 @@
  * This class holds reference to the C++ EventEmitter object. Instances of this class are created on
  * the Bindings.cpp, where the pointer to the C++ event emitter is set.
  */
+@Nullsafe(Nullsafe.Mode.LOCAL)
 @SuppressLint("MissingNativeLoadLibrary")
 @DoNotStrip
 public class StateWrapperImpl implements StateWrapper {

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

@@ -11,13 +11,15 @@
 import static com.facebook.react.fabric.mounting.LayoutMetricsConversions.getMinSize;
 
 import androidx.annotation.IntDef;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 import com.facebook.react.bridge.NativeMap;
 import com.facebook.react.interfaces.fabric.SurfaceHandler;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 
+@Nullsafe(Nullsafe.Mode.LOCAL)
 public class SurfaceHandlerBinding implements SurfaceHandler {
   static {
     FabricSoLoader.staticInit();

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/fabric/events/EventBeatManager.java

@@ -9,6 +9,7 @@
 
 import android.annotation.SuppressLint;
 import androidx.annotation.NonNull;
+import com.facebook.infer.annotation.Nullsafe;
 import com.facebook.jni.HybridData;
 import com.facebook.proguard.annotations.DoNotStrip;
 import com.facebook.react.bridge.ReactApplicationContext;
@@ -18,6 +19,7 @@
 /**
  * Class that acts as a proxy between the list of EventBeats registered in C++ and the Android side.
  */
+@Nullsafe(Nullsafe.Mode.LOCAL)
 @SuppressLint("MissingNativeLoadLibrary")
 public final class EventBeatManager implements BatchEventDispatchedListener {