fix WritableArray, WritableMap nullable annotations

ReactAndroid/src/main/java/com/facebook/react/bridge/JavaOnlyArray.java

@@ -160,7 +160,7 @@ public void pushInt(int value) {
   }
 
   @Override
-  public void pushString(@Nonnull String value) {
+  public void pushString(@Nullable String value) {
     mBackingList.add(value);
   }
 
@@ -170,7 +170,7 @@ public void pushArray(@Nullable WritableArray array) {
   }
 
   @Override
-  public void pushMap(@Nonnull WritableMap map) {
+  public void pushMap(@Nullable WritableMap map) {
     mBackingList.add(map);
   }
 

ReactAndroid/src/main/java/com/facebook/react/bridge/JavaOnlyMap.java

@@ -12,6 +12,7 @@
 import java.util.Map;
 
 import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
 
 /**
  * Java {@link HashMap} backed implementation of {@link ReadableMap} and {@link WritableMap}
@@ -179,7 +180,7 @@ public void putInt(@Nonnull String key, int value) {
   }
 
   @Override
-  public void putString(@Nonnull String key, @Nonnull String value) {
+  public void putString(@Nonnull String key, @Nullable String value) {
     mBackingMap.put(key, value);
   }
 
@@ -189,7 +190,7 @@ public void putNull(@Nonnull String key) {
   }
 
   @Override
-  public void putMap(@Nonnull String key, @Nonnull WritableMap value) {
+  public void putMap(@Nonnull String key, @Nullable WritableMap value) {
     mBackingMap.put(key, value);
   }
 
@@ -199,7 +200,7 @@ public void merge(@Nonnull ReadableMap source) {
   }
 
   @Override
-  public void putArray(@Nonnull String key, @Nonnull WritableArray value) {
+  public void putArray(@Nonnull String key, @Nullable WritableArray value) {
     mBackingMap.put(key, value);
   }
 

ReactAndroid/src/main/java/com/facebook/react/bridge/WritableArray.java

@@ -8,6 +8,7 @@
 package com.facebook.react.bridge;
 
 import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
 
 /**
  * Interface for a mutable array. Used to pass arguments from Java to JS.
@@ -18,7 +19,7 @@ public interface WritableArray extends ReadableArray {
   void pushBoolean(boolean value);
   void pushDouble(double value);
   void pushInt(int value);
-  void pushString(@Nonnull String value);
-  void pushArray(@Nonnull WritableArray array);
-  void pushMap(@Nonnull WritableMap map);
+  void pushString(@Nullable String value);
+  void pushArray(@Nullable WritableArray array);
+  void pushMap(@Nullable WritableMap map);
 }

ReactAndroid/src/main/java/com/facebook/react/bridge/WritableMap.java

@@ -8,6 +8,7 @@
 package com.facebook.react.bridge;
 
 import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
 
 /**
  * Interface for a mutable map. Used to pass arguments from Java to JS.
@@ -18,9 +19,9 @@ public interface WritableMap extends ReadableMap {
   void putBoolean(@Nonnull String key, boolean value);
   void putDouble(@Nonnull String key, double value);
   void putInt(@Nonnull String key, int value);
-  void putString(@Nonnull String key, @Nonnull String value);
-  void putArray(@Nonnull String key, @Nonnull WritableArray value);
-  void putMap(@Nonnull String key, @Nonnull WritableMap value);
+  void putString(@Nonnull String key, @Nullable String value);
+  void putArray(@Nonnull String key, @Nullable WritableArray value);
+  void putMap(@Nonnull String key, @Nullable WritableMap value);
 
   void merge(@Nonnull ReadableMap source);
 }

ReactAndroid/src/main/java/com/facebook/react/bridge/WritableNativeArray.java

@@ -38,7 +38,7 @@ public WritableNativeArray() {
   @Override
   public native void pushInt(int value);
   @Override
-  public native void pushString(@Nonnull String value);
+  public native void pushString(@Nullable String value);
 
   // Note: this consumes the map so do not reuse it.
   @Override
@@ -50,7 +50,7 @@ public void pushArray(@Nullable WritableArray array) {
 
   // Note: this consumes the map so do not reuse it.
   @Override
-  public void pushMap(@Nonnull WritableMap map) {
+  public void pushMap(@Nullable WritableMap map) {
     Assertions.assertCondition(
         map == null || map instanceof WritableNativeMap, "Illegal type provided");
     pushNativeMap((WritableNativeMap) map);

ReactAndroid/src/main/java/com/facebook/react/bridge/WritableNativeMap.java

@@ -14,6 +14,7 @@
 import com.facebook.proguard.annotations.DoNotStrip;
 
 import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
 
 /**
  * Implementation of a write-only map stored in native memory. Use
@@ -33,21 +34,21 @@ public class WritableNativeMap extends ReadableNativeMap implements WritableMap
   @Override
   public native void putInt(@Nonnull String key, int value);
   @Override
-  public native void putString(@Nonnull String key, @Nonnull String value);
+  public native void putString(@Nonnull String key, @Nullable String value);
   @Override
   public native void putNull(@NonNull String key);
 
   // Note: this consumes the map so do not reuse it.
   @Override
-  public void putMap(@Nonnull String key, @Nonnull WritableMap value) {
+  public void putMap(@Nonnull String key, @Nullable WritableMap value) {
     Assertions.assertCondition(
         value == null || value instanceof WritableNativeMap, "Illegal type provided");
     putNativeMap(key, (WritableNativeMap) value);
   }
 
   // Note: this consumes the map so do not reuse it.
   @Override
-  public void putArray(@Nonnull String key, @Nonnull WritableArray value) {
+  public void putArray(@Nonnull String key, @Nullable WritableArray value) {
     Assertions.assertCondition(
         value == null || value instanceof WritableNativeArray, "Illegal type provided");
     putNativeArray(key, (WritableNativeArray) value);