inspector: move process.binding to internalBinding

In places of process.binding('inspector'), migrate code to adapt
internalBinding.

PR-URL: #24931
Refs: #22160
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>

Backport-PR-URL: #25446

Author: BeniCheni

lib/inspector.js

@@ -11,7 +11,7 @@ const {
 } = require('internal/errors').codes;
 const { validateString } = require('internal/validators');
 const util = require('util');
-const { Connection, open, url } = process.binding('inspector');
+const { Connection, open, url } = internalBinding('inspector');
 
 if (!Connection)
   throw new ERR_INSPECTOR_NOT_AVAILABLE();

lib/internal/bootstrap/loaders.js

@@ -90,6 +90,7 @@ const internalBindingWhitelist = [
   'fs_event_wrap',
   'http_parser',
   'icu',
+  'inspector',
   'js_stream',
   'natives',
   'pipe_wrap',

lib/internal/console/inspector.js

@@ -4,7 +4,7 @@ const path = require('path');
 const CJSModule = require('internal/modules/cjs/loader');
 const { makeRequireFunction } = require('internal/modules/cjs/helpers');
 const { tryGetCwd } = require('internal/util');
-const { addCommandLineAPI, consoleCall } = process.binding('inspector');
+const { addCommandLineAPI, consoleCall } = internalBinding('inspector');
 
 // Wrap a console implemented by Node.js with features from the VM inspector
 function addInspectorApis(consoleFromNode, consoleFromVM) {

lib/internal/inspector_async_hook.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const inspector = process.binding('inspector');
+const inspector = internalBinding('inspector');
 
 if (!inspector || !inspector.asyncTaskScheduled) {
   exports.setup = function() {};

lib/internal/modules/cjs/helpers.js

@@ -108,7 +108,7 @@ if (getOptionValue('--experimental-worker')) {
   builtinLibs.sort();
 }
 
-if (typeof process.binding('inspector').open === 'function') {
+if (typeof internalBinding('inspector').open === 'function') {
   builtinLibs.push('inspector');
   builtinLibs.sort();
 }

lib/internal/modules/cjs/loader.js

@@ -706,7 +706,7 @@ Module.prototype._compile = function(content, filename) {
     // Set breakpoint on module start
     if (filename === resolvedArgv) {
       delete process._breakFirstLine;
-      inspectorWrapper = process.binding('inspector').callAndPauseOnStart;
+      inspectorWrapper = internalBinding('inspector').callAndPauseOnStart;
     }
   }
   var dirname = path.dirname(filename);

lib/internal/modules/esm/module_job.js

@@ -73,7 +73,7 @@ class ModuleJob {
     try {
       if (this.isMain && process._breakFirstLine) {
         delete process._breakFirstLine;
-        const initWrapper = process.binding('inspector').callAndPauseOnStart;
+        const initWrapper = internalBinding('inspector').callAndPauseOnStart;
         initWrapper(this.module.instantiate, this.module);
       } else {
         this.module.instantiate();

lib/internal/process/coverage.js

@@ -51,7 +51,7 @@ function disableAllAsyncHooks() {
 exports.writeCoverage = writeCoverage;
 
 function setup() {
-  const { Connection } = process.binding('inspector');
+  const { Connection } = internalBinding('inspector');
   if (!Connection) {
     console.warn('inspector not enabled');
     return;

src/inspector_js_api.cc

@@ -315,5 +315,5 @@ void Initialize(Local<Object> target, Local<Value> unused,
 }  // namespace inspector
 }  // namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(inspector,
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(inspector,
                                   node::inspector::Initialize);

src/node.cc

@@ -2138,5 +2138,5 @@ int Start(int argc, char** argv) {
 #if !HAVE_INSPECTOR
 void Initialize() {}
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(inspector, Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(inspector, Initialize)
 #endif  // !HAVE_INSPECTOR

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

@@ -17,3 +17,4 @@ assert(process.binding('spawn_sync'));
 assert(process.binding('js_stream'));
 assert(process.binding('buffer'));
 assert(process.binding('fs'));
+assert(process.binding('inspector'));

test/sequential/test-async-wrap-getasyncid.js

@@ -291,7 +291,7 @@ if (common.hasCrypto) { // eslint-disable-line node-core/crypto-check
 
 if (process.config.variables.v8_enable_inspector !== 0 &&
     common.isMainThread) {
-  const binding = process.binding('inspector');
+  const binding = internalBinding('inspector');
   const handle = new binding.Connection(() => {});
   testInitialized(handle, 'Connection');
   handle.disconnect();