Convert com.facebook.react.bridge.ReadableNativeArray to Kotlin (facebook#47484)

Summary:
Pull Request resolved: facebook#47484

Changelog: [Internal]

Differential Revision: D65596278

Author: javache

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

@@ -1482,7 +1482,7 @@ public class com/facebook/react/bridge/ReadableNativeArray : com/facebook/react/
 	public fun getDouble (I)D
 	public fun getDynamic (I)Lcom/facebook/react/bridge/Dynamic;
 	public fun getInt (I)I
-	public static fun getJNIPassCounter ()I
+	public static final fun getJNIPassCounter ()I
 	public fun getLong (I)J
 	public synthetic fun getMap (I)Lcom/facebook/react/bridge/ReadableMap;
 	public fun getMap (I)Lcom/facebook/react/bridge/ReadableNativeMap;

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

@@ -130,7 +130,7 @@ public class InterpolationAnimatedNode(config: ReadableMap) : ValueAnimatedNode(
       val outputRange = arrayOfNulls<DoubleArray>(size)
 
       // Match the first pattern into a List, since we don't know its length yet
-      var m = numericPattern.matcher(array.getString(0))
+      var m = numericPattern.matcher(array.getString(0) ?: "")
       val firstOutputRange: MutableList<Double> = ArrayList()
       while (m.find()) {
         firstOutputRange.add(m.group().toDouble())
@@ -143,7 +143,7 @@ public class InterpolationAnimatedNode(config: ReadableMap) : ValueAnimatedNode(
       for (i in 1 until size) {
         val outputArr = DoubleArray(firstOutputRangeArr.size)
         var j = 0
-        m = numericPattern.matcher(array.getString(i))
+        m = numericPattern.matcher(array.getString(i) ?: "")
         while (m.find() && j < firstOutputRangeArr.size) {
           outputArr[j++] = m.group().toDouble()
         }

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

@@ -45,7 +45,9 @@ internal class ObjectAnimatedNode(
         ReadableType.String -> result.pushString(source.getString(i))
         ReadableType.Map -> {
           val map = source.getMap(i)
-          if (map.hasKey(NODE_TAG_KEY) && map.getType(NODE_TAG_KEY) == ReadableType.Number) {
+          if (map != null &&
+              map.hasKey(NODE_TAG_KEY) &&
+              map.getType(NODE_TAG_KEY) == ReadableType.Number) {
             val node = nativeAnimatedNodesManager.getNodeById(map.getInt(NODE_TAG_KEY))
             requireNotNull(node) { "Mapped value node does not exist" }
             if (node is ValueAnimatedNode) {

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

@@ -19,15 +19,15 @@ internal class TransformAnimatedNode(
     config: ReadableMap,
     private val nativeAnimatedNodesManager: NativeAnimatedNodesManager
 ) : AnimatedNode() {
-  private val transformConfigs: MutableList<TransformConfig>
+  private val transformConfigs: List<TransformConfig>
 
   init {
     val transforms = config.getArray("transforms")
     transformConfigs =
-        if (transforms == null) mutableListOf()
+        if (transforms == null) emptyList()
         else
-            MutableList<TransformConfig>(transforms.size()) { i ->
-              val transformConfigMap = transforms.getMap(i)
+            List<TransformConfig>(transforms.size()) { i ->
+              val transformConfigMap = checkNotNull(transforms.getMap(i))
               val property = transformConfigMap.getString("property")
               val type = transformConfigMap.getString("type")
               if (type == "animated") {
@@ -46,7 +46,7 @@ internal class TransformAnimatedNode(
 
   public fun collectViewUpdates(propsMap: JavaOnlyMap) {
     val transforms =
-        MutableList<JavaOnlyMap>(transformConfigs.size) { i ->
+        List<JavaOnlyMap>(transformConfigs.size) { i ->
           val transformConfig = transformConfigs[i]
           val transform =
               if (transformConfig is AnimatedTransformConfig) {

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

@@ -34,7 +34,7 @@ private static boolean supportWideGamut() {
   private static final String ATTR_SEGMENT = "attr/";
 
   @Nullable
-  private static Integer getColorInteger(Object value, Context context) {
+  private static Integer getColorInteger(@Nullable Object value, Context context) {
     if (value == null) {
       return null;
     }
@@ -84,7 +84,7 @@ private static Integer getColorInteger(Object value, Context context) {
   }
 
   @Nullable
-  public static Color getColorInstance(Object value, Context context) {
+  public static Color getColorInstance(@Nullable Object value, Context context) {
     if (value == null) {
       return null;
     }
@@ -136,7 +136,7 @@ public static Color getColorInstance(Object value, Context context) {
         "ColorValue: the value must be a number or Object.");
   }
 
-  public static Integer getColor(Object value, Context context) {
+  public static Integer getColor(@Nullable Object value, Context context) {
     try {
       if (supportWideGamut()) {
         Color color = getColorInstance(value, context);
@@ -150,7 +150,7 @@ public static Integer getColor(Object value, Context context) {
     return getColorInteger(value, context);
   }
 
-  public static Integer getColor(Object value, Context context, int defaultInt) {
+  public static Integer getColor(@Nullable Object value, Context context, int defaultInt) {
     try {
       return getColor(value, context);
     } catch (JSApplicationCausedNativeException e) {

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

@@ -73,13 +73,13 @@ public class JavaOnlyArray : ReadableArray, WritableArray {
 
   override fun getLong(index: Int): Long = (backingList[index] as Number).toLong()
 
-  override fun getString(index: Int): String = backingList[index] as String
+  override fun getString(index: Int): String? = backingList[index] as String?
 
-  override fun getArray(index: Int): ReadableArray = backingList[index] as ReadableArray
+  override fun getArray(index: Int): ReadableArray? = backingList[index] as ReadableArray?
 
   override fun getBoolean(index: Int): Boolean = backingList[index] as Boolean
 
-  override fun getMap(index: Int): ReadableMap = backingList[index] as ReadableMap
+  override fun getMap(index: Int): ReadableMap? = backingList[index] as ReadableMap?
 
   override fun getDynamic(index: Int): Dynamic = DynamicFromArray.create(this, index)
 

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

@@ -14,7 +14,7 @@ import java.util.ArrayList
  * to Kotlin.
  */
 public interface ReadableArray {
-  public fun getArray(index: Int): ReadableArray
+  public fun getArray(index: Int): ReadableArray?
 
   public fun getBoolean(index: Int): Boolean
 
@@ -26,9 +26,9 @@ public interface ReadableArray {
 
   public fun getLong(index: Int): Long
 
-  public fun getMap(index: Int): ReadableMap
+  public fun getMap(index: Int): ReadableMap?
 
-  public fun getString(index: Int): String
+  public fun getString(index: Int): String?
 
   public fun getType(index: Int): ReadableType