0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

LibWeb: Support passing more parameter types to HTML::report_exception()

Browse source 
We now allow any JS::ThrowCompletion<T>, as well as JS::Completion
directly (although we'll VERIFY() that it's a throw completion.)
This commit is contained in:
Andreas Kling 2022-02-08 15:37:36 +01:00 • committed by Linus Groh
parent 3383ae1598
commit 4db5406d62
Notes: sideshowbarker 2024-07-17 19:09:32 +09:00
2 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibWeb/HTML/Scripting/ExceptionReporter.cpp
View file

@ -12,12 +12,13 @@
namespace Web::HTML {
// https://html.spec.whatwg.org/#report-the-exception
void report_exception(JS::ThrowCompletionOr<JS::Value> const& result)
void report_exception(JS::Completion const& throw_completion)
{
// FIXME: This is just old code, and does not strictly follow the spec of report an exception.
// FIXME: We should probably also report these exceptions to the JS console.
VERIFY(result.throw_completion().value().has_value());
auto thrown_value = *result.throw_completion().value();
VERIFY(throw_completion.type() == JS::Completion::Type::Throw);
VERIFY(throw_completion.value().has_value());
auto thrown_value = *throw_completion.value();
if (thrown_value.is_object()) {
auto& object = thrown_value.as_object();
auto& vm = object.vm();

9
Userland/Libraries/LibWeb/HTML/Scripting/ExceptionReporter.h
View file

@ -10,6 +10,13 @@
namespace Web::HTML {
void report_exception(JS::ThrowCompletionOr<JS::Value> const& value);
void report_exception(JS::Completion const&);
template<typename T>
inline void report_exception(JS::ThrowCompletionOr<T> const& result)
{
VERIFY(result.is_throw_completion());
report_exception(result.throw_completion());
}
}