child_process: move process.binding('spawn_sync') to internalBinding

PR-URL: #22260 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: John-David Dalton <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Sep 15, 2018 · 9c9c01f · 9c9c01f
1 parent dadd6e1
commit 9c9c01f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
@@ -376,7 +376,8 @@
         'tls_wrap',
         'util',
         'async_wrap',
-        'url']);
+        'url',
+        'spawn_sync']);
     process.binding = function binding(name) {
       return internalBindingWhitelist.has(name) ?
         internalBinding(name) :

diff --git a/lib/internal/child_process.js b/lib/internal/child_process.js
@@ -22,7 +22,6 @@ const util = require('util');
 const assert = require('assert');
 
 const { internalBinding } = require('internal/bootstrap/loaders');
-
 const { Process } = internalBinding('process_wrap');
 const { WriteWrap } = internalBinding('stream_wrap');
 const { Pipe, constants: PipeConstants } = internalBinding('pipe_wrap');
@@ -33,7 +32,7 @@ const SocketList = require('internal/socket_list');
 const { owner_symbol } = require('internal/async_hooks').symbols;
 const { convertToValidSignal } = require('internal/util');
 const { isArrayBufferView } = require('internal/util/types');
-const spawn_sync = process.binding('spawn_sync');
+const spawn_sync = internalBinding('spawn_sync');
 const { HTTPParser } = internalBinding('http_parser');
 const { freeParser } = require('_http_common');
 const { kStateSymbol } = require('internal/dgram');

diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc
@@ -1100,5 +1100,5 @@ void SyncProcessRunner::KillTimerCloseCallback(uv_handle_t* handle) {
 
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(spawn_sync,
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(spawn_sync,
   node::SyncProcessRunner::Initialize)
diff --git a/test/parallel/test-process-binding-internalbinding-whitelist.js b/test/parallel/test-process-binding-internalbinding-whitelist.js
@@ -13,3 +13,4 @@ assert(process.binding('stream_wrap'));
 assert(process.binding('signal_wrap'));
 assert(process.binding('contextify'));
 assert(process.binding('url'));
+assert(process.binding('spawn_sync'));