Do not check for navigator to detect web environment in built-in extensions (#251688)

Do not check for navigator to detect web environment

Author: jeanp413

extensions/simple-browser/src/extension.ts

@@ -87,5 +87,5 @@ export function activate(context: vscode.ExtensionContext) {
 
 function isWeb(): boolean {
 	// @ts-expect-error
-	return typeof navigator !== 'undefined' && vscode.env.uiKind === vscode.UIKind.Web;
+	return !(typeof process === 'object' && !!process.versions.node) && vscode.env.uiKind === vscode.UIKind.Web;
 }

extensions/typescript-language-features/src/utils/platform.ts

@@ -6,7 +6,7 @@
 import * as vscode from 'vscode';
 
 export function isWeb(): boolean {
-	return 'navigator' in globalThis && vscode.env.uiKind === vscode.UIKind.Web;
+	return !(typeof process === 'object' && !!process.versions.node) && vscode.env.uiKind === vscode.UIKind.Web;
 }
 
 export function isWebAndHasSharedArrayBuffers(): boolean {
@@ -16,4 +16,3 @@ export function isWebAndHasSharedArrayBuffers(): boolean {
 export function supportsReadableByteStreams(): boolean {
 	return isWeb() && 'ReadableByteStreamController' in globalThis;
 }
-