Revert of Make Error.prototype.toString spec compliant; and fix various side-effect-free error printing metho…

src/api.cc

Index: src/api.cc
diff --git a/src/api.cc b/src/api.cc
index ddd87d5c41a885de27e5a2b60339d53c3856bab5..38d641dd99bfac3280f4b6185d74d44e2d9a042c 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -2898,16 +2898,12 @@
 
 
 MaybeLocal<String> Value::ToDetailString(Local<Context> context) const {
-  i::Handle<i::Object> obj = Utils::OpenHandle(this);
+  auto obj = Utils::OpenHandle(this);
   if (obj->IsString()) return ToApiHandle<String>(obj);
   PREPARE_FOR_EXECUTION(context, "ToDetailString", String);
   Local<String> result;
-  i::Handle<i::Object> args[] = {obj};
-  has_pending_exception = !ToLocal<String>(
-      i::Execution::TryCall(isolate, isolate->no_side_effects_to_string_fun(),
-                            isolate->factory()->undefined_value(),
-                            arraysize(args), args),
-      &result);
+  has_pending_exception =
+      !ToLocal<String>(i::Execution::ToDetailString(isolate, obj), &result);
   RETURN_ON_FAILED_EXECUTION(String);
   RETURN_ESCAPED(result);
 }