Browser shouldn't steal window focus (#311082)

Co-authored-by: Copilot <copilot@github.com>

Author: kycutler

src/vs/platform/browserView/common/browserView.ts

@@ -326,8 +326,9 @@ export interface IBrowserViewService {
 	/**
 	 * Focus the browser view
 	 * @param id The browser view identifier
+	 * @param force Whether to force focus even if the view's window is not focused.
 	 */
-	focus(id: string): Promise<void>;
+	focus(id: string, force?: boolean): Promise<void>;
 
 	/**
 	 * Find text in the browser view's page

src/vs/platform/browserView/electron-main/browserView.ts

@@ -594,7 +594,11 @@ export class BrowserView extends Disposable {
 	/**
 	 * Focus this view
 	 */
-	async focus(): Promise<void> {
+	async focus(force?: boolean): Promise<void> {
+		// By default, only focus the view if its window is already focused.
+		if (!force && !this._window?.win?.isFocused()) {
+			return;
+		}
 		this._view.webContents.focus();
 	}
 

src/vs/platform/browserView/electron-main/browserViewMainService.ts

@@ -202,8 +202,8 @@ export class BrowserViewMainService extends Disposable implements IBrowserViewMa
 		return this._getBrowserView(id).captureScreenshot(options);
 	}
 
-	async focus(id: string): Promise<void> {
-		return this._getBrowserView(id).focus();
+	async focus(id: string, force?: boolean): Promise<void> {
+		return this._getBrowserView(id).focus(force);
 	}
 
 	async findInPage(id: string, text: string, options?: IBrowserViewFindInPageOptions): Promise<void> {

src/vs/workbench/contrib/browserView/common/browserView.ts

@@ -203,7 +203,7 @@ export interface IBrowserViewModel extends IDisposable {
 	reload(hard?: boolean): Promise<void>;
 	toggleDevTools(): Promise<void>;
 	captureScreenshot(options?: IBrowserViewCaptureScreenshotOptions): Promise<VSBuffer>;
-	focus(): Promise<void>;
+	focus(force?: boolean): Promise<void>;
 	findInPage(text: string, options?: IBrowserViewFindInPageOptions): Promise<void>;
 	stopFindInPage(keepSelection?: boolean): Promise<void>;
 	getSelectedText(): Promise<string>;
@@ -492,8 +492,8 @@ export class BrowserViewModel extends Disposable implements IBrowserViewModel {
 		return result;
 	}
 
-	async focus(): Promise<void> {
-		return this.browserViewService.focus(this.id);
+	async focus(force?: boolean): Promise<void> {
+		return this.browserViewService.focus(this.id, force);
 	}
 
 	async findInPage(text: string, options?: IBrowserViewFindInPageOptions): Promise<void> {