mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-11 03:56:16 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

IDLGenerators: Support string types in setlike

Browse source
This commit is contained in:
Sam Atkins 2025-07-04 13:58:31 +01:00 • committed by Tim Ledbetter
commit efcbec250f
Notes: github-actions[bot] 2025-07-04 17:12:16 +00:00
1 changed files with 22 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

32
Meta/Lagom/Tools/CodeGenerators/LibWeb/BindingsGenerator/IDLGenerators.cpp
View file

@ -4529,7 +4529,25 @@ JS_DEFINE_NATIVE_FUNCTION(@class_name@::values)
if (interface.set_entry_type.has_value()) { if (interface.set_entry_type.has_value()) {
auto setlike_generator = generator.fork(); auto setlike_generator = generator.fork();
setlike_generator.set("value_type", interface.set_entry_type.value()->name()); auto const& set_entry_type = *interface.set_entry_type.value();
setlike_generator.set("value_type", set_entry_type.name());
if (set_entry_type.is_string()) {
setlike_generator.set("value_type_check", R"~~~(
if (!value_arg.is_string()) {
return vm.throw_completion<JS::TypeError>(JS::ErrorType::NotAnObjectOfType, "String");
}
)~~~");
} else {
setlike_generator.set("value_type_check",
MUST(String::formatted(R"~~~(
if (!value_arg.is_object() || !is<{0}>(value_arg.as_object())) {{
return vm.throw_completion<JS::TypeError>(JS::ErrorType::NotAnObjectOfType, "{0}");
}}
)~~~",
set_entry_type.name())));
}
setlike_generator.append(R"~~~( setlike_generator.append(R"~~~(
// https://webidl.spec.whatwg.org/#js-set-size // https://webidl.spec.whatwg.org/#js-set-size
JS_DEFINE_NATIVE_FUNCTION(@class_name@::get_size) JS_DEFINE_NATIVE_FUNCTION(@class_name@::get_size)
@ -4595,9 +4613,7 @@ JS_DEFINE_NATIVE_FUNCTION(@class_name@::has)
GC::Ref<JS::Set> set = impl->set_entries(); GC::Ref<JS::Set> set = impl->set_entries();
auto value_arg = vm.argument(0); auto value_arg = vm.argument(0);
if (!value_arg.is_object() || !is<@value_type@>(value_arg.as_object())) { @value_type_check@
return vm.throw_completion<JS::TypeError>(JS::ErrorType::NotAnObjectOfType, "@value_type@");
}
// FIXME: If value is -0, set value to +0. // FIXME: If value is -0, set value to +0.
// What? Which interfaces have a number as their set type? // What? Which interfaces have a number as their set type?
@ -4617,9 +4633,7 @@ JS_DEFINE_NATIVE_FUNCTION(@class_name@::add)
GC::Ref<JS::Set> set = impl->set_entries(); GC::Ref<JS::Set> set = impl->set_entries();
auto value_arg = vm.argument(0); auto value_arg = vm.argument(0);
if (!value_arg.is_object() || !is<@value_type@>(value_arg.as_object())) { @value_type_check@
return vm.throw_completion<JS::TypeError>(JS::ErrorType::NotAnObjectOfType, "@value_type@");
}
// FIXME: If value is -0, set value to +0. // FIXME: If value is -0, set value to +0.
// What? Which interfaces have a number as their set type? // What? Which interfaces have a number as their set type?
@ -4641,9 +4655,7 @@ JS_DEFINE_NATIVE_FUNCTION(@class_name@::delete_)
GC::Ref<JS::Set> set = impl->set_entries(); GC::Ref<JS::Set> set = impl->set_entries();
auto value_arg = vm.argument(0); auto value_arg = vm.argument(0);
if (!value_arg.is_object() || !is<@value_type@>(value_arg.as_object())) { @value_type_check@
return vm.throw_completion<JS::TypeError>(JS::ErrorType::NotAnObjectOfType, "@value_type@");
}
// FIXME: If value is -0, set value to +0. // FIXME: If value is -0, set value to +0.
// What? Which interfaces have a number as their set type? // What? Which interfaces have a number as their set type?