Fix assertion failure in spawn-related tests (#4400)

* Clean up some of the event loop code

* Support timeouts

* Defer freeing FilePoll

---------

Co-authored-by: Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/src/bun.js/web_worker.zig b/src/bun.js/web_worker.zig
index e9c9637b9..b4de8aad2 100644
--- a/src/bun.js/web_worker.zig
+++ b/src/bun.js/web_worker.zig
@@ -263,8 +263,7 @@ pub const WebWorker = struct {
         this.setStatus(.running);
 
         // don't run the GC if we don't actually need to
-        if (vm.eventLoop().tasks.count > 0 or vm.active_tasks > 0 or
-            vm.event_loop_handle.?.active > 0 or
+        if (vm.isEventLoopAlive() or
             vm.eventLoop().tickConcurrentWithCount() > 0)
         {
             vm.global.vm().releaseWeakRefs();
@@ -275,7 +274,7 @@ pub const WebWorker = struct {
         // always doing a first tick so we call CppTask without delay after dispatchOnline
         vm.tick();
 
-        while (vm.eventLoop().tasks.count > 0 or vm.active_tasks > 0 or vm.event_loop_handle.?.active > 0) {
+        while (vm.isEventLoopAlive()) {
             vm.tick();
             if (this.requested_terminate) break;
             vm.eventLoop().autoTickActive();