Remove web debugging support (#15350)

* Remove web debugging support

* format

* Change files

* Remove remote-debugging option from cli

Author: acoates-ms

.spelling

@@ -851,7 +851,6 @@ WDIO
 WEAKPTR
 WEAKREFCOUNT
 weakthis
-webdebugger
 webdriverio
 webhosthidden
 Wexceptions

change/@react-native-windows-cli-61eae078-8c0c-4af3-b0e7-bcc3c5c192d0.json

@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "Remove web debugging support",
+  "packageName": "@react-native-windows/cli",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/react-native-windows-e65abdb1-63c6-4fdd-85c5-1ad95131cfa4.json

@@ -0,0 +1,7 @@
+{
+  "type": "prerelease",
+  "comment": "Remove web debugging support",
+  "packageName": "react-native-windows",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

docs/inspector.md

@@ -11,7 +11,6 @@ Hence, we expect most tooling available on other platforms to just work on Windo
 1. Initialize React Native Host,
    - Turn on `DeveloperSupport` 
    - Turn on `FastRefresh`
-   - Turn off `WebDebugger`
    - Turn on `Direct Debugging`
 2. Ensure Dev-Server is running
 3. Start the application

packages/@react-native-windows/cli/README.md

@@ -93,7 +93,6 @@ Here are the options that `npx @react-native-community/cli run-windows` takes:
 | `--emulator`          | boolean    | Deploys the app to an emulator                   |
 | `--device`            | boolean    | Deploys the app to a connected device            |
 | `--target`            | string     | Deploys the app to the specified `GUID` for a device |
-| `--remote-debugging`  | boolean    | **(Deprecated)** Deploys the app in remote debugging mode |
 | `--logging`           | boolean    | Verbose output logging                           |
 | `--no-packager`       | boolean    | Do not launch the packager while building        |
 | `--bundle`            | boolean    | Enable Bundle configuration, i.e. `ReleaseBundle`/`DebugBundle` rather than `Release`/`Debug` |

packages/@react-native-windows/cli/src/commands/runWindows/runWindows.ts

@@ -59,7 +59,6 @@ function optionSanitizer(key: keyof RunWindowsOptions, value: any): any {
     case 'singleproc':
     case 'emulator':
     case 'device':
-    case 'remoteDebugging':
     case 'logging':
     case 'packager':
     case 'bundle':

packages/@react-native-windows/cli/src/commands/runWindows/runWindowsOptions.ts

@@ -19,7 +19,6 @@ export type BuildConfig = 'Debug' | 'DebugBundle' | 'Release' | 'ReleaseBundle';
  *    emulator: Boolean - Deploy to the emulator
  *    device: Boolean - Deploy to a device
  *    target: String - Device GUID to deploy to
- *    remote-debugging: Boolean - Run using remote JS proxy
  *    logging: Boolean - Enables logging
  *    no-packager: Boolean - Do not launch packager while building
  *    bundle: Boolean - Enable Bundle configuration.
@@ -40,7 +39,6 @@ export interface RunWindowsOptions {
   emulator?: boolean;
   device?: boolean;
   target?: string;
-  remoteDebugging?: string;
   logging?: boolean;
   packager?: boolean;
   bundle?: boolean;
@@ -91,10 +89,6 @@ export const runWindowsOptions: CommandOption[] = [
     name: '--target [string]',
     description: 'Deploys the app to the specified GUID for a device',
   },
-  {
-    name: '--remote-debugging',
-    description: '(Deprecated) Deploys the app in remote debugging mode',
-  },
   {
     name: '--logging',
     description: 'Verbose output logging',

packages/@react-native-windows/cli/src/e2etest/runWindows.test.ts

@@ -25,7 +25,6 @@ function validateOptionName(
     case 'emulator':
     case 'device':
     case 'target':
-    case 'remoteDebugging':
     case 'logging':
     case 'packager':
     case 'bundle':

packages/@react-native-windows/cli/src/utils/deploy.ts

@@ -380,9 +380,6 @@ export async function deployToDesktop(
   const vsVersion = Version.fromString(buildTools.installationVersion);
 
   const args = [];
-  if (options.remoteDebugging) {
-    args.push('--remote-debugging');
-  }
 
   if (options.directDebugging) {
     args.push('--direct-debugging', options.directDebugging.toString());

packages/debug-test/DebuggingFeatures.test.ts

@@ -65,7 +65,6 @@ test('debug target properties', async () => {
   testLog.message(`executing 'debug target properties' on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });
@@ -110,7 +109,6 @@ test('enable, disable', async () => {
   testLog.message(`executing 'enable, disable' test on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });
@@ -135,7 +133,6 @@ test('pause, resume', async () => {
   testLog.message(`executing 'pause, resume' test on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });
@@ -198,7 +195,6 @@ test('set, remove breakpoint', async () => {
   testLog.message(`executing 'set, remove breakpoint' test on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });
@@ -245,7 +241,6 @@ test('reload after continue', async () => {
   testLog.message(`executing 'pause, resume' test on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });
@@ -284,7 +279,6 @@ test('execution context identifier in Debugger.scriptParsed event', async () =>
   testLog.message(`executing 'pause, resume' test on PID ${pid}`);
 
   const settings = await PlaygroundDebugSettings.set({
-    webDebugger: false,
     directDebugging: true,
     jsEngine: 'Hermes',
   });