Back out "Update ColorPropConverterto support color function values" (facebook#44225)

Summary:
Pull Request resolved: facebook#44225

Back out [facebook#43031](facebook#43031) as it makes some internal test fails.

## Changelog:
[Android][Fixed] - Revert facebook#43031

Reviewed By: arushikesarwani94

Differential Revision: D56489260

fbshipit-source-id: bd843e6eb60fce0a1d5d8da9f3c5d2a36473ecfb

Author: cipolleschi

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

@@ -649,7 +649,6 @@ public class com/facebook/react/bridge/ColorPropConverter {
 	public fun <init> ()V
 	public static fun getColor (Ljava/lang/Object;Landroid/content/Context;)Ljava/lang/Integer;
 	public static fun getColor (Ljava/lang/Object;Landroid/content/Context;I)Ljava/lang/Integer;
-	public static fun getColorInstance (Ljava/lang/Object;Landroid/content/Context;)Landroid/graphics/Color;
 	public static fun resolveResourcePath (Landroid/content/Context;Ljava/lang/String;)Ljava/lang/Integer;
 }
 

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

@@ -9,11 +9,7 @@
 
 import android.content.Context;
 import android.content.res.Resources;
-import android.graphics.Color;
-import android.graphics.ColorSpace;
-import android.os.Build;
 import android.util.TypedValue;
-import androidx.annotation.ColorLong;
 import androidx.annotation.Nullable;
 import androidx.core.content.res.ResourcesCompat;
 import com.facebook.common.logging.FLog;
@@ -28,17 +24,13 @@ public class ColorPropConverter {
   private static final String ATTR = "attr";
   private static final String ATTR_SEGMENT = "attr/";
 
-  private static Boolean apiSupportWideGamut() {
-    return Build.VERSION.SDK_INT >= Build.VERSION_CODES.O;
-  }
-
-  public static Color getColorInstance(Object value, Context context) {
+  public static Integer getColor(Object value, Context context) {
     if (value == null) {
       return null;
     }
 
-    if (apiSupportWideGamut() && value instanceof Double) {
-      return Color.valueOf(((Double) value).intValue());
+    if (value instanceof Double) {
+      return ((Double) value).intValue();
     }
 
     if (context == null) {
@@ -47,22 +39,6 @@ public static Color getColorInstance(Object value, Context context) {
 
     if (value instanceof ReadableMap) {
       ReadableMap map = (ReadableMap) value;
-
-      // handle color(space r g b a) value
-      if (apiSupportWideGamut() && map.hasKey("space")) {
-        String rawColorSpace = map.getString("space");
-        boolean isDisplayP3 = rawColorSpace.equals("display-p3");
-        ColorSpace space =
-            ColorSpace.get(isDisplayP3 ? ColorSpace.Named.DISPLAY_P3 : ColorSpace.Named.SRGB);
-        float r = (float) map.getDouble("r");
-        float g = (float) map.getDouble("g");
-        float b = (float) map.getDouble("b");
-        float a = (float) map.getDouble("a");
-
-        @ColorLong long color = Color.pack(r, g, b, a, space);
-        return Color.valueOf(color);
-      }
-
       ReadableArray resourcePaths = map.getArray(JSON_KEY);
 
       if (resourcePaths == null) {
@@ -72,8 +48,8 @@ public static Color getColorInstance(Object value, Context context) {
 
       for (int i = 0; i < resourcePaths.size(); i++) {
         Integer result = resolveResourcePath(context, resourcePaths.getString(i));
-        if (apiSupportWideGamut() && result != null) {
-          return Color.valueOf(result);
+        if (result != null) {
+          return result;
         }
       }
 
@@ -87,17 +63,6 @@ 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) {
-    Color color = getColorInstance(value, context);
-    if (color == null) {
-      return null;
-    }
-    if (apiSupportWideGamut()) {
-      return color.toArgb();
-    }
-    return null;
-  }
-
   public static Integer getColor(Object value, Context context, int defaultInt) {
     try {
       return getColor(value, context);
@@ -124,9 +89,8 @@ public static Integer resolveResourcePath(Context context, @Nullable String reso
         return resolveThemeAttribute(context, resourcePath);
       }
     } catch (Resources.NotFoundException exception) {
-      // The resource could not be found so do nothing to allow the for loop to
-      // continue and
-      // try the next fallback resource in the array. If none of the fallbacks are
+      // The resource could not be found so do nothing to allow the for loop to continue and
+      // try the next fallback resource in the array.  If none of the fallbacks are
       // found then the exception immediately after the for loop will be thrown.
     }
     return null;