Revert "child_process: change windowsHide default to true"

This reverts commit 420d8afe3db22ad703e74892f58f9e32d34ff699.

PR-URL: https://github.com/nodejs/node/pull/24034
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>

1 files changed, 2 insertions, 4 deletions

diff --git a/lib/child_process.js b/lib/child_process.js
index e2b7a32920..711764dba7 100644
--- a/lib/child_process.js
+++ b/lib/child_process.js
@@ -224,7 +224,7 @@ exports.execFile = function execFile(file /* , args, options, callback */) {
     gid: options.gid,
     uid: options.uid,
     shell: options.shell,
-    windowsHide: options.windowsHide !== false,
+    windowsHide: !!options.windowsHide,
     windowsVerbatimArguments: !!options.windowsVerbatimArguments
   });
 
@@ -544,7 +544,7 @@ var spawn = exports.spawn = function spawn(/* file, args, options */) {
     file: opts.file,
     args: opts.args,
     cwd: options.cwd,
-    windowsHide: options.windowsHide !== false,
+    windowsHide: !!options.windowsHide,
     windowsVerbatimArguments: !!options.windowsVerbatimArguments,
     detached: !!options.detached,
     envPairs: opts.envPairs,
@@ -578,8 +578,6 @@ function spawnSync(/* file, args, options */) {
 
   options.stdio = _validateStdio(options.stdio || 'pipe', true).stdio;
 
-  options.windowsHide = options.windowsHide !== false;
-
   if (options.input) {
     var stdin = options.stdio[0] = util._extend({}, options.stdio[0]);
     stdin.input = options.input;