From 747b79d91585f90338a28eb0868b4d317d504e25 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 1 Apr 2026 01:01:41 +0000 Subject: [PATCH] Fix "target closed" error in BrowserViewDebugger by checking isDestroyed before detach Agent-Logs-Url: https://github.com/microsoft/vscode/sessions/8233ab57-121b-4677-9a2b-aee47e824ba6 Co-authored-by: bryanchen-d <41454397+bryanchen-d@users.noreply.github.com> --- .../browserView/electron-main/browserViewDebugger.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/vs/platform/browserView/electron-main/browserViewDebugger.ts b/src/vs/platform/browserView/electron-main/browserViewDebugger.ts index ebdbdb3bf3e..26208f9a464 100644 --- a/src/vs/platform/browserView/electron-main/browserViewDebugger.ts +++ b/src/vs/platform/browserView/electron-main/browserViewDebugger.ts @@ -168,6 +168,13 @@ export class BrowserViewDebugger extends Disposable implements ICDPTarget { } this._electronDebugger.removeListener('message', this._messageHandler); + + // If the WebContents is already destroyed, there is nothing to detach from. + // This happens when disposal is triggered by the 'destroyed' event on the WebContents. + if (this.view.webContents.isDestroyed()) { + return; + } + try { this._electronDebugger.detach(); } catch (error) {