chore: fix the errors

puppeteer · Apr 23, 2024 · 7ba043c · 7ba043c
1 parent 8dfbffb
commit 7ba043c
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 7 deletions.
diff --git a/packages/puppeteer-core/src/bidi/CDPSession.ts b/packages/puppeteer-core/src/bidi/CDPSession.ts
@@ -21,7 +21,7 @@ export class BidiCdpSession extends CDPSession {
   static sessions = new Map<string, BidiCdpSession>();
 
   #detached = false;
-  readonly #connection: BidiConnection | undefined = undefined;
+  readonly #connection?: BidiConnection;
   readonly #sessionId = Deferred.create<string>();
   readonly frame: BidiFrame;
 
@@ -41,11 +41,11 @@ export class BidiCdpSession extends CDPSession {
     } else {
       (async () => {
         try {
-          const session = await connection.send('cdp.getSession', {
+          const {result} = await connection.send('cdp.getSession', {
             context: frame._id,
           });
-          this.#sessionId.resolve(session.result.session!);
-          BidiCdpSession.sessions.set(session.result.session!, this);
+          this.#sessionId.resolve(result.session!);
+          BidiCdpSession.sessions.set(result.session!, this);
         } catch (error) {
           this.#sessionId.reject(error as Error);
         }
@@ -89,13 +89,29 @@ export class BidiCdpSession extends CDPSession {
   }
 
   override async detach(): Promise<void> {
-    if (this.#connection === undefined || this.#detached) {
+    if (
+      this.#connection === undefined ||
+      this.#connection.closed ||
+      this.#detached
+    ) {
       return;
     }
+    try {
+      await this.frame.client.send('Target.detachFromTarget', {
+        sessionId: this.id(),
+      });
+    } finally {
+      this.onClose();
+    }
+  }
 
+  /**
+   * @internal
+   */
+  onClose = (): void => {
     BidiCdpSession.sessions.delete(this.id());
     this.#detached = true;
-  }
+  };
 
   override id(): string {
     const value = this.#sessionId.value();

diff --git a/packages/puppeteer-core/src/bidi/Frame.ts b/packages/puppeteer-core/src/bidi/Frame.ts
@@ -106,7 +106,7 @@ export class BidiFrame extends Frame {
     this.browsingContext.on('closed', () => {
       for (const session of BidiCdpSession.sessions.values()) {
         if (session.frame === this) {
-          void session.detach().catch(debugError);
+          session.onClose();
         }
       }
       this.page().trustedEmitter.emit(PageEvent.FrameDetached, this);