Fabric: Additional temporary checks in prop parsing infra

Summary:
While ViewConfig infra isn't perfect we need to check some value for correctness during prop-parsing.

Changelog: [Internal] Fabric-specific internal change.

Reviewed By: JoshuaGross

Differential Revision: D20639055

fbshipit-source-id: 193dcd0769bc7777bc8d60c964ede72ebdaa83e4

ReactCommon/fabric/components/image/conversions.h

@@ -52,23 +52,35 @@ inline void fromRawValue(const RawValue &value, ImageSource &result) {
       result.scale = items.find("deprecated") != items.end() ? 0.0 : 1.0;
     }
 
-    if (items.find("url") != items.end()) {
+    if (items.find("url") != items.end() &&
+        // The following should be removed after codegen is shipped.
+        // See T45151459.
+        items.at("url").hasType<std::string>()) {
       result.uri = (std::string)items.at("url");
     }
 
-    if (items.find("uri") != items.end()) {
+    if (items.find("uri") != items.end() &&
+        // The following should be removed after codegen is shipped.
+        // See T45151459.
+        items.at("uri").hasType<std::string>()) {
       result.uri = (std::string)items.at("uri");
     }
 
-    if (items.find("bundle") != items.end()) {
+    if (items.find("bundle") != items.end() &&
+        // The following should be removed after codegen is shipped.
+        // See T45151459.
+        items.at("bundle").hasType<std::string>()) {
       result.bundle = (std::string)items.at("bundle");
       result.type = ImageSource::Type::Local;
     }
 
     return;
   }
 
-  abort();
+  // The following should be removed after codegen is shipped.
+  // See T45151459.
+  result = {};
+  result.type = ImageSource::Type::Invalid;
 }
 
 inline std::string toString(const ImageSource &value) {