mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-12-24 23:23:58 +00:00
LibWeb: Let various functions throw if not enough arguments
...instead of handing out null / undefined / empty values.
This commit is contained in:
parent
2d503b20da
commit
e3e9749d88
Notes:
sideshowbarker
2024-07-19 06:17:19 +09:00
Author: https://github.com/linusg Commit: https://github.com/SerenityOS/serenity/commit/e3e9749d884 Pull-request: https://github.com/SerenityOS/serenity/pull/2318
3 changed files with 9 additions and 8 deletions
|
@ -76,7 +76,7 @@ JS::Value DocumentWrapper::get_element_by_id(JS::Interpreter& interpreter)
|
|||
if (!document)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return JS::js_null();
|
||||
return interpreter.throw_exception<JS::TypeError>("getElementById() needs one argument");
|
||||
auto id = interpreter.argument(0).to_string(interpreter);
|
||||
if (interpreter.exception())
|
||||
return {};
|
||||
|
@ -92,7 +92,7 @@ JS::Value DocumentWrapper::query_selector_all(JS::Interpreter& interpreter)
|
|||
if (!document)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return JS::js_null();
|
||||
return interpreter.throw_exception<JS::TypeError>("querySelectorAll() needs one argument");
|
||||
auto selector = interpreter.argument(0).to_string(interpreter);
|
||||
if (interpreter.exception())
|
||||
return {};
|
||||
|
|
|
@ -27,6 +27,7 @@
|
|||
#include <AK/FlyString.h>
|
||||
#include <AK/Function.h>
|
||||
#include <LibJS/Interpreter.h>
|
||||
#include <LibJS/Runtime/Error.h>
|
||||
#include <LibJS/Runtime/Function.h>
|
||||
#include <LibJS/Runtime/GlobalObject.h>
|
||||
#include <LibWeb/Bindings/EventListenerWrapper.h>
|
||||
|
@ -54,7 +55,7 @@ JS::Value EventTargetWrapper::add_event_listener(JS::Interpreter& interpreter)
|
|||
if (!this_object)
|
||||
return {};
|
||||
if (interpreter.argument_count() < 2)
|
||||
return JS::js_undefined();
|
||||
return interpreter.throw_exception<JS::TypeError>("addEventListener() needs two arguments");
|
||||
auto event_name = interpreter.argument(0).to_string(interpreter);
|
||||
if (interpreter.exception())
|
||||
return {};
|
||||
|
|
|
@ -128,8 +128,8 @@ JS::Value WindowObject::set_interval(JS::Interpreter& interpreter)
|
|||
auto* impl = impl_from(interpreter);
|
||||
if (!impl)
|
||||
return {};
|
||||
if (interpreter.argument_count() < 2)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return interpreter.throw_exception<JS::TypeError>("setInterval() needs at least one argument");
|
||||
auto* callback_object = interpreter.argument(0).to_object(interpreter);
|
||||
if (!callback_object)
|
||||
return {};
|
||||
|
@ -148,7 +148,7 @@ JS::Value WindowObject::set_timeout(JS::Interpreter& interpreter)
|
|||
if (!impl)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return {};
|
||||
return interpreter.throw_exception<JS::TypeError>("setTimeout() needs at least one argument");
|
||||
auto* callback_object = interpreter.argument(0).to_object(interpreter);
|
||||
if (!callback_object)
|
||||
return {};
|
||||
|
@ -172,7 +172,7 @@ JS::Value WindowObject::request_animation_frame(JS::Interpreter& interpreter)
|
|||
if (!impl)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return {};
|
||||
return interpreter.throw_exception<JS::TypeError>("requestAnimationFrame() needs one argument");
|
||||
auto* callback_object = interpreter.argument(0).to_object(interpreter);
|
||||
if (!callback_object)
|
||||
return {};
|
||||
|
@ -187,7 +187,7 @@ JS::Value WindowObject::cancel_animation_frame(JS::Interpreter& interpreter)
|
|||
if (!impl)
|
||||
return {};
|
||||
if (!interpreter.argument_count())
|
||||
return {};
|
||||
return interpreter.throw_exception<JS::TypeError>("cancelAnimationFrame() needs one argument");
|
||||
auto id = interpreter.argument(0).to_i32(interpreter);
|
||||
if (interpreter.exception())
|
||||
return {};
|
||||
|
|
Loading…
Reference in a new issue