extension host - add reason property and adopt (#180514)

src/vs/workbench/contrib/relauncher/browser/relauncher.contribution.ts

@@ -199,7 +199,7 @@ export class WorkspaceChangeExtHostRelauncher extends Disposable implements IWor
 			if (environmentService.remoteAuthority) {
 				hostService.reload(); // TODO@aeschli, workaround
 			} else if (isNative) {
-				const stopped = await extensionService.stopExtensionHosts();
+				const stopped = await extensionService.stopExtensionHosts(localize('restartExtensionHost.reason', "Restart of extensions required because of workspace folder change."));
 				if (stopped) {
 					extensionService.startExtensionHosts();
 				}

src/vs/workbench/services/extensions/common/abstractExtensionService.ts

@@ -621,14 +621,14 @@ export abstract class AbstractExtensionService extends Disposable implements IEx
 
 	//#region Stopping / Starting / Restarting
 
-	public stopExtensionHosts(): Promise<boolean>;
+	public stopExtensionHosts(reason: string): Promise<boolean>;
 	public stopExtensionHosts(force: true): void;
-	public stopExtensionHosts(force?: boolean): void | Promise<boolean> {
-		if (force) {
+	public stopExtensionHosts(arg0: true | string): void | Promise<boolean> {
+		if (arg0 === true) {
 			return this._doStopExtensionHosts();
 		}
 
-		return this._doStopExtensionHostsWithVeto();
+		return this._doStopExtensionHostsWithVeto(arg0);
 	}
 
 	protected _doStopExtensionHosts(): void {
@@ -655,10 +655,11 @@ export abstract class AbstractExtensionService extends Disposable implements IEx
 		}
 	}
 
-	private async _doStopExtensionHostsWithVeto(): Promise<boolean> {
+	private async _doStopExtensionHostsWithVeto(reason: string): Promise<boolean> {
 		const vetos: (boolean | Promise<boolean>)[] = [];
 
 		this._onWillStop.fire({
+			reason,
 			veto(value) {
 				vetos.push(value);
 			}

src/vs/workbench/services/extensions/common/extensions.ts

@@ -325,6 +325,13 @@ export const enum ActivationKind {
 
 export interface WillStopExtensionHostsEvent {
 
+	/**
+	 * A human readable reason for stopping the extension hosts
+	 * that e.g. can be shown in a confirmation dialog to the
+	 * user.
+	 */
+	readonly reason: string;
+
 	/**
 	 * Allows to veto the stopping of extension hosts. The veto can be a long running
 	 * operation.
@@ -445,10 +452,13 @@ export interface IExtensionService {
 	/**
 	 * Stops the extension hosts.
 	 *
+	 * @param reason a human readable reason for stopping the extension hosts. This maybe
+	 * can be presented to the user when showing dialogs.
+	 *
 	 * @returns a promise that resolves to `true` if the extension hosts were stopped, `false`
 	 * if the operation was vetoed by listeners of the `onWillStop` event.
 	 */
-	stopExtensionHosts(): Promise<boolean>;
+	stopExtensionHosts(reason: string): Promise<boolean>;
 
 	/**
 	 * @deprecated Use `stopExtensionHosts()` instead.

src/vs/workbench/services/extensions/electron-sandbox/nativeExtensionService.ts

@@ -720,7 +720,7 @@ class RestartExtensionHostAction extends Action2 {
 	async run(accessor: ServicesAccessor): Promise<void> {
 		const extensionService = accessor.get(IExtensionService);
 
-		const stopped = await extensionService.stopExtensionHosts();
+		const stopped = await extensionService.stopExtensionHosts(nls.localize('restartExtensionHost.reason', "Restart of extensions explicitly requested by user."));
 		if (stopped) {
 			extensionService.startExtensionHosts();
 		}

src/vs/workbench/services/workspaces/electron-sandbox/workspaceEditingService.ts

@@ -154,7 +154,7 @@ export class NativeWorkspaceEditingService extends AbstractWorkspaceEditingServi
 	}
 
 	async enterWorkspace(workspaceUri: URI): Promise<void> {
-		const stopped = await this.extensionService.stopExtensionHosts();
+		const stopped = await this.extensionService.stopExtensionHosts(localize('restartExtensionHost.reason', "Restart of extensions required because of opening a multi-root workspace."));
 		if (!stopped) {
 			return;
 		}