Andreas Kling bc6e61adec LibWeb: Don't churn HTML::EventLoop while in microtask checkpoint ...

At the end of HTML::EventLoop::process(), the loop reschedules itself if
there are more runnable tasks available.

However, the condition was flawed: we would reschedule if there were any
microtasks queued, but those tasks will not be processed if we're
currently within the scope of a microtask checkpoint.

To fix this, we now only reschedule the HTML event loop for microtask
processing *if* we're not already in a microtask checkpoint.

This fixes the 100% CPU churn seen when looking at PRs on GitHub. :^)

2023-04-01 12:45:47 +01:00

19 KiB

Raw Blame History

View Raw