LibJS: Convert to_string() to ThrowCompletionOr

Also update get_function_name() to use ThrowCompletionOr, but this is
not a standard AO and should be refactored out of existence eventually.

1 files changed, 2 insertions, 6 deletions

diff --git a/Userland/Libraries/LibJS/Runtime/ErrorConstructor.cpp b/Userland/Libraries/LibJS/Runtime/ErrorConstructor.cpp
index c108374734..1f0ed8a418 100644
--- a/Userland/Libraries/LibJS/Runtime/ErrorConstructor.cpp
+++ b/Userland/Libraries/LibJS/Runtime/ErrorConstructor.cpp
@@ -42,9 +42,7 @@ Value ErrorConstructor::construct(FunctionObject& new_target)
     auto* error = TRY_OR_DISCARD(ordinary_create_from_constructor<Error>(global_object, new_target, &GlobalObject::error_prototype));
 
     if (!vm.argument(0).is_undefined()) {
-        auto message = vm.argument(0).to_string(global_object);
-        if (vm.exception())
-            return {};
+        auto message = TRY_OR_DISCARD(vm.argument(0).to_string(global_object));
         MUST(error->create_non_enumerable_data_property_or_throw(vm.names.message, js_string(vm, message)));
     }
 
@@ -89,9 +87,7 @@ Value ErrorConstructor::construct(FunctionObject& new_target)
             global_object, new_target, &GlobalObject::snake_name##_prototype));                                  \
                                                                                                                  \
         if (!vm.argument(0).is_undefined()) {                                                                    \
-            auto message = vm.argument(0).to_string(global_object);                                              \
-            if (vm.exception())                                                                                  \
-                return {};                                                                                       \
+            auto message = TRY_OR_DISCARD(vm.argument(0).to_string(global_object));                              \
             MUST(error->create_non_enumerable_data_property_or_throw(vm.names.message, js_string(vm, message))); \
         }                                                                                                        \
                                                                                                                  \