Don't wait for browser.close() to prevent hanging - fire and forget approach

Co-authored-by: kobenguyent <7845001+kobenguyent@users.noreply.github.com>

Author: Copilot

lib/helper/Playwright.js

@@ -1389,29 +1389,7 @@ class Playwright extends Helper {
       }
     }
 
-    // Close all contexts and pages before closing browser to prevent hanging
-    if (this.browser) {
-      try {
-        const contexts = await Promise.race([
-          this.browser.contexts(),
-          new Promise((_, reject) => setTimeout(() => reject(new Error('Get contexts timeout')), 1000))
-        ])
-        // Close all pages in all contexts first
-        await Promise.allSettled(contexts.map(async (ctx) => {
-          try {
-            const pages = await ctx.pages()
-            await Promise.allSettled(pages.map(p => p.close().catch(() => {})))
-          } catch (e) {
-            // Ignore errors getting or closing pages
-          }
-        }))
-        // Then close all contexts
-        await Promise.allSettled(contexts.map(c => c.close().catch(() => {})))
-      } catch (e) {
-        // Ignore errors if browser is already closed or timeout getting contexts
-      }
-    }
-
+    // Close browserContext if recordHar is enabled
     if (this.options.recordHar && this.browserContext) {
       try {
         await this.browserContext.close()
@@ -1421,16 +1399,16 @@ class Playwright extends Helper {
     }
     this.browserContext = null
 
+    // Initiate browser close without waiting for it to complete
+    // The browser process will be cleaned up when the Node process exits
     if (this.browser) {
       try {
-        // Add timeout to prevent browser.close() from hanging indefinitely
-        await Promise.race([this.browser.close(), new Promise((_, reject) => setTimeout(() => reject(new Error('Browser close timeout')), 5000))])
+        // Fire and forget - don't wait for close to complete
+        this.browser.close().catch(() => {
+          // Silently ignore any errors during async close
+        })
       } catch (e) {
-        // Ignore errors if browser is already closed or timeout
-        if (!e.message?.includes('Browser close timeout')) {
-          // Non-timeout error, can be ignored as well
-        }
-        // Force cleanup even on error
+        // Ignore any synchronous errors
       }
     }
     this.browser = null