fix(Playwright): Properly clean up event listeners to prevent hanging

- Remove all page event listeners (crash, dialog, load, console, requestfinished) in _setPage
- Remove all browser event listeners in _stopBrowser before closing
- Add error handling for listener cleanup to prevent failures
- Wrap browser.close() and browserContext.close() in try-catch blocks

This should help prevent the Node.js event loop from staying alive due to
uncleaned event listeners, though additional investigation may be needed
for complete resolution of process hanging after tests complete.

Author: kobenguyent

lib/helper/Playwright.js

@@ -1056,6 +1056,9 @@ class Playwright extends Helper {
       try {
         this.page.removeAllListeners('crash')
         this.page.removeAllListeners('dialog')
+        this.page.removeAllListeners('load')
+        this.page.removeAllListeners('console')
+        this.page.removeAllListeners('requestfinished')
       } catch (e) {
         console.warn('Warning cleaning previous page listeners:', e.message)
       }
@@ -1284,9 +1287,32 @@ class Playwright extends Helper {
     this.context = null
     this.frame = null
     popupStore.clear()
-    if (this.options.recordHar) await this.browserContext.close()
+    
+    // Remove all event listeners to prevent hanging
+    if (this.browser) {
+      try {
+        this.browser.removeAllListeners()
+      } catch (e) {
+        // Ignore errors if browser is already closed
+      }
+    }
+    
+    if (this.options.recordHar && this.browserContext) {
+      try {
+        await this.browserContext.close()
+      } catch (e) {
+        // Ignore errors if context is already closed
+      }
+    }
     this.browserContext = null
-    await this.browser.close()
+    
+    if (this.browser) {
+      try {
+        await this.browser.close()
+      } catch (e) {
+        // Ignore errors if browser is already closed
+      }
+    }
     this.browser = null
     this.isRunning = false
   }