fix: always close file handles to stdout/stderr logs

Author: Junyan

src/chrome-launcher.ts

@@ -405,11 +405,6 @@ class Launcher {
 
   destroyTmp() {
     return new Promise<void>(resolve => {
-      // Only clean up the tmp dir if we created it.
-      if (this.userDataDir === undefined || this.opts.userDataDir !== undefined) {
-        return resolve();
-      }
-
       if (this.outFile) {
         this.fs.closeSync(this.outFile);
         delete this.outFile;
@@ -420,6 +415,11 @@ class Launcher {
         delete this.errFile;
       }
 
+      // Only clean up the tmp dir if we created it.
+      if (this.userDataDir === undefined || this.opts.userDataDir !== undefined) {
+        return resolve();
+      }
+
       // backwards support for node v12 + v14.14+
       // https://nodejs.org/api/deprecations.html#DEP0147
       const rm = this.fs.rm || this.fs.rmdir;