chore(bidi): create as Bidi session before browser.close

Author: yury-s

packages/playwright-core/src/server/bidi/bidiConnection.ts

@@ -28,7 +28,8 @@ import type * as bidi from './third_party/bidiProtocol';
 
 // BidiPlaywright uses this special id to issue Browser.close command which we
 // should ignore.
-export const kBrowserCloseMessageId = 0;
+export const kBrowserCloseMessageId = Number.MAX_SAFE_INTEGER - 1;
+export const kShutdownSessionNewMessageId = kBrowserCloseMessageId - 1;
 
 export class BidiConnection {
   private readonly _transport: ConnectionTransport;
@@ -209,7 +210,7 @@ export class BidiSession extends EventEmitter {
 
   dispatchMessage(message: any) {
     const object = message as bidi.Message;
-    if (object.id === kBrowserCloseMessageId)
+    if (object.id === kBrowserCloseMessageId || object.id === kShutdownSessionNewMessageId)
       return;
     if (object.id && this._callbacks.has(object.id)) {
       const callback = this._callbacks.get(object.id)!;

packages/playwright-core/src/server/bidi/bidiFirefox.ts

@@ -20,7 +20,7 @@ import path from 'path';
 import { wrapInASCIIBox } from '../utils/ascii';
 import { BrowserType, kNoXServerRunningError } from '../browserType';
 import { BidiBrowser } from './bidiBrowser';
-import { kBrowserCloseMessageId } from './bidiConnection';
+import { kBrowserCloseMessageId, kShutdownSessionNewMessageId } from './bidiConnection';
 import { createProfile } from './third_party/firefoxPrefs';
 import { ManualPromise } from '../../utils/isomorphic/manualPromise';
 
@@ -78,6 +78,17 @@ export class BidiFirefox extends BrowserType {
   }
 
   override attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
+    // browser.close does not work without an active session, make sure
+    // to create a new session first. The command will fail if a session
+    // already exists, we'll just ignore the error.
+    transport.send({ method: 'session.new', params: {
+      capabilities: {
+        alwaysMatch: {
+          unhandledPromptBehavior: { default: 'ignore' },
+          webSocketUrl: true
+        }
+      }
+    }, id: kShutdownSessionNewMessageId });
     transport.send({ method: 'browser.close', params: {}, id: kBrowserCloseMessageId });
   }