custom userDataDir log file error

closeSync outFile and errFile when userDataDir is custom.

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;
@@ -419,6 +414,11 @@ class Launcher {
         this.fs.closeSync(this.errFile);
         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