LibJS: Remove implicit wrapping/unwrapping of completion records

This is an editorial change in the ECMA-262 spec, with similar changes
in some proposals.

See:
- https://github.com/tc39/ecma262/commit/7575f74
- https://github.com/tc39/proposal-array-grouping/commit/df899eb
- https://github.com/tc39/proposal-shadowrealm/commit/9eb5a12
- https://github.com/tc39/proposal-shadowrealm/commit/c81f527

1 files changed, 6 insertions, 6 deletions

diff --git a/Userland/Libraries/LibJS/CyclicModule.h b/Userland/Libraries/LibJS/CyclicModule.h
index 5d87aa1958..2f66b5d977 100644
--- a/Userland/Libraries/LibJS/CyclicModule.h
+++ b/Userland/Libraries/LibJS/CyclicModule.h
@@ -37,13 +37,13 @@ protected:
     virtual ThrowCompletionOr<u32> inner_module_linking(VM& vm, Vector<Module*>& stack, u32 index) override;
     virtual ThrowCompletionOr<u32> inner_module_evaluation(VM& vm, Vector<Module*>& stack, u32 index) override;
 
-    virtual Completion initialize_environment(VM& vm);
-    virtual Completion execute_module(VM& vm, Optional<PromiseCapability> capability = {});
+    virtual ThrowCompletionOr<void> initialize_environment(VM& vm);
+    virtual ThrowCompletionOr<void> execute_module(VM& vm, Optional<PromiseCapability> capability = {});
 
-    ThrowCompletionOr<void> execute_async_module(VM& vm);
-    ThrowCompletionOr<void> gather_available_ancestors(Vector<CyclicModule*>& exec_list);
-    ThrowCompletionOr<void> async_module_execution_fulfilled(VM& vm);
-    ThrowCompletionOr<void> async_module_execution_rejected(VM& vm, Value error);
+    void execute_async_module(VM& vm);
+    void gather_available_ancestors(Vector<CyclicModule*>& exec_list);
+    void async_module_execution_fulfilled(VM& vm);
+    void async_module_execution_rejected(VM& vm, Value error);
 
     ModuleStatus m_status { ModuleStatus::Unlinked };   // [[Status]]
     ThrowCompletionOr<void> m_evaluation_error;         // [[EvaluationError]]