Render Error objects in CDT (#1471)

Summary:
X-link: facebook/react-native#45990

Pull Request resolved: #1471

Changelog:
[General][Added]: support for rendering Error objects in Chrome DevTools console

As discussed in [Linkifying and symbolicating JavaScript Error stacks in Fusebox](https://docs.google.com/document/d/1JI_PPzxFRwRNii6pcx-4Cb7g8UTrqnZHCxaoynntIrM/edit) by hoxyq, Error objects in CDT currently displays a big blob of string.

In this diff, we send the correct CDT params.

Reviewed By: hoxyq

Differential Revision: D60598446

fbshipit-source-id: 527c4e5858dfb879a32b40e5ad4df73a22e694e5

API/hermes/cdp/RemoteObjectConverters.cpp

@@ -18,6 +18,13 @@ namespace m = ::facebook::hermes::cdp::message;
 
 constexpr size_t kMaxPreviewProperties = 10;
 
+static bool isObjectInstanceOfError(
+    const jsi::Object &obj,
+    facebook::jsi::Runtime &runtime) {
+  return obj.instanceOf(
+      runtime, runtime.global().getPropertyAsFunction(runtime, "Error"));
+}
+
 static m::runtime::PropertyPreview generatePropertyPreview(
     facebook::jsi::Runtime &runtime,
     const std::string &name,
@@ -309,6 +316,16 @@ m::runtime::RemoteObject m::runtime::makeRemoteObject(
       if (options.generatePreview) {
         result.preview = generateArrayPreview(runtime, array);
       }
+    } else if (isObjectInstanceOfError(obj, runtime)) {
+      result.type = "object";
+      result.subtype = "error";
+      // T198854404 we should report subclasses of Error here, e.g. TypeError
+      result.className = "Error";
+      result.description =
+          obj.getProperty(runtime, "stack").toString(runtime).utf8(runtime);
+      if (options.generatePreview) {
+        result.preview = generateObjectPreview(runtime, obj);
+      }
     } else {
       result.type = "object";
       result.description = result.className = "Object";