LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 01:37:39 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Do not pass JS::Handle into NativeFunction callback captures

Browse Source 
NativeFunction uses JS::HeapFunction for the callback so GC-allocated
captures will be visited anyway.
This commit is contained in:
Aliaksandr Kalenik 2023-09-26 16:47:29 +02:00 committed by Andreas Kling
parent 90aaa8ab40
commit 3bcd368703
Notes: sideshowbarker 2024-07-18 03:20:18 +09:00 
Author: https://github.com/kalenikaliaksandr
Commit: https://github.com/SerenityOS/serenity/commit/3bcd368703
Pull-request: https://github.com/SerenityOS/serenity/pull/21235
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Userland/Libraries/LibWeb/DOM/EventTarget.cpp
View File

@ -561,7 +561,7 @@ void EventTarget::activate_event_handler(FlyString const& name, HTML::EventHandl
// location.reload();
// The body element is no longer in the DOM and there is no variable holding onto it. However, the onunload handler is still called, meaning the callback keeps the body element alive.
auto callback_function = JS::NativeFunction::create(
realm, [event_target = JS::make_handle(*this), name](JS::VM& vm) mutable -> JS::ThrowCompletionOr<JS::Value> {
realm, [event_target = this, name](JS::VM& vm) mutable -> JS::ThrowCompletionOr<JS::Value> {
// The event dispatcher should only call this with one argument.
VERIFY(vm.argument_count() == 1);