mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-12 20:42:21 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb: Use the global object to access the performance object

Browse source 
Previously, we were accessing the performance through the current
window object. Thus caused a crash when `animate()` was called on an
element within a document with no associated window object. The global
object is now used to access the performance object in places where
a window object is not guaranteed to exist.
This commit is contained in:
Tim Ledbetter 2024-03-30 10:04:31 +00:00 • committed by Andreas Kling
commit 558fef237c
Notes: sideshowbarker 2024-07-17 11:33:34 +09:00
3 changed files with 15 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/WebAnimations/misc/animate-no-window.txt Normal file
View file

@ -0,0 +1 @@
PASS (didn't crash)

11
Tests/LibWeb/Text/input/WebAnimations/misc/animate-no-window.html Normal file
View file

@ -0,0 +1,11 @@
<!DOCTYPE html>
<script src="../../include.js"></script>
<script>
test(() => {
const divElement = document.createElement("div");
const newDocument = document.implementation.createDocument("http://www.w3.org/1999/xhtml", "html");
newDocument.documentElement.appendChild(divElement)
animation = divElement.animate({}, {});
println("PASS (didn't crash)");
});
</script>

5
Userland/Libraries/LibWeb/DOM/Document.cpp
View file

@ -4185,8 +4185,9 @@ void Document::ensure_animation_timer()
m_animation_driver_timer->stop(); m_animation_driver_timer->stop();
return; return;
} }
auto* window_or_worker = dynamic_cast<HTML::WindowOrWorkerGlobalScopeMixin*>(&realm().global_object());
update_animations_and_send_events(window()->performance()->now()); VERIFY(window_or_worker);
update_animations_and_send_events(window_or_worker->performance()->now());
})); }));
} }