From 18c735429f9fd5475e779e2ee6c74fcbbb9ecf00 Mon Sep 17 00:00:00 2001 From: Daniel Bertalan Date: Thu, 29 Jun 2023 08:28:15 +0200 Subject: [PATCH] LibJS: Fix incorrect `is_function()` check in NewPromiseCapability I introduced this copy-paste mistake in c21255d. Gets rid of 7 assertion failures on test262. --- Userland/Libraries/LibJS/Runtime/PromiseCapability.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibJS/Runtime/PromiseCapability.cpp b/Userland/Libraries/LibJS/Runtime/PromiseCapability.cpp index b964ba6bcf..dc82e2d894 100644 --- a/Userland/Libraries/LibJS/Runtime/PromiseCapability.cpp +++ b/Userland/Libraries/LibJS/Runtime/PromiseCapability.cpp @@ -96,7 +96,7 @@ ThrowCompletionOr> new_promise_capability(VM& vm return vm.throw_completion(ErrorType::NotAFunction, "Promise capability resolve value"); // 8. If IsCallable(resolvingFunctions.[[Reject]]) is false, throw a TypeError exception. - if (!resolving_functions->resolve.is_function()) + if (!resolving_functions->reject.is_function()) return vm.throw_completion(ErrorType::NotAFunction, "Promise capability reject value"); // 9. Return the PromiseCapability Record { [[Promise]]: promise, [[Resolve]]: resolvingFunctions.[[Resolve]], [[Reject]]: resolvingFunctions.[[Reject]] }.