cherry-pick(#30135): chore: do not exit UI mode upon page reload

pavelfeldman · pavelfeldman · commit 75776dbc3e15 · 2024-03-26T15:51:25.000-07:00
diff --git a/packages/playwright-core/src/server/trace/viewer/traceViewer.ts b/packages/playwright-core/src/server/trace/viewer/traceViewer.ts
@@ -229,7 +229,6 @@ class StdinServer implements Transport {
   }
 
   onclose() {
-    gracefullyProcessExitDoNotHang(0);
   }
 
   sendEvent?: (method: string, params: any) => void;
diff --git a/packages/playwright/src/runner/testServer.ts b/packages/playwright/src/runner/testServer.ts
@@ -98,7 +98,7 @@ class TestServerDispatcher implements TestServerInterface {
     this._configFile = configFile;
     this.transport = {
       dispatch: (method, params) => (this as any)[method](params),
-      onclose: () => gracefullyProcessExitDoNotHang(0),
+      onclose: () => {},
     };
     this._globalWatcher = new Watcher('deep', () => this._dispatchEvent('listChanged', {}));
     this._testWatcher = new Watcher('flat', events => {
@@ -392,6 +392,7 @@ async function innerRunTestServer(configFile: string | undefined, options: { hos
   const testServer = new TestServer(configFile);
   const cancelPromise = new ManualPromise<void>();
   const sigintWatcher = new SigIntWatcher();
+  process.stdin.on('close', () => gracefullyProcessExitDoNotHang(0));
   void sigintWatcher.promise().then(() => cancelPromise.resolve());
   try {
     const server = await testServer.start(options);

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,6 @@ class StdinServer implements Transport {`
`229`	`229`	`}`
`230`	`230`
`231`	`231`	`onclose() {`
`232`		`- gracefullyProcessExitDoNotHang(0);`
`233`	`232`	`}`
`234`	`233`
`235`	`234`	`sendEvent?: (method: string, params: any) => void;`