v8: move process.binding('v8') to internalBinding

PR-URL: #22288
Refs: #22160
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jon Moss <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>

lib/internal/bootstrap/node.js

@@ -344,7 +344,7 @@
     // that are whitelisted for access via process.binding()... this is used
     // to provide a transition path for modules that are being moved over to
     // internalBinding.
-    const internalBindingWhitelist = new SafeSet(['uv', 'http_parser']);
+    const internalBindingWhitelist = new SafeSet(['uv', 'http_parser', 'v8']);
     process.binding = function binding(name) {
       return internalBindingWhitelist.has(name) ?
         internalBinding(name) :

lib/v8.js

@@ -56,7 +56,7 @@ const {
   kSpaceUsedSizeIndex,
   kSpaceAvailableSizeIndex,
   kPhysicalSpaceSizeIndex
-} = process.binding('v8');
+} = internalBinding('v8');
 
 const kNumberOfHeapSpaces = kHeapSpaces.length;
 

src/node_v8.cc

@@ -200,4 +200,4 @@ void Initialize(Local<Object> target,
 
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(v8, node::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(v8, node::Initialize)

test/parallel/test-process-binding-internalbinding-whitelist.js

@@ -8,3 +8,4 @@ const assert = require('assert');
 // process.binding().
 assert(process.binding('uv'));
 assert(process.binding('http_parser'));
+assert(process.binding('v8'));