fixup! src: use internalBinding('config').hasInspector in JS land

nodejs · Jan 2, 2019 · b249b07 · b249b07
1 parent ac1a309
commit b249b07
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/internal/bootstrap/cache.js b/lib/internal/bootstrap/cache.js
@@ -33,7 +33,7 @@ const cannotUseCache = [
 
 // Skip modules that cannot be required when they are not
 // built into the binary.
-if (hasInspector) {
+if (!hasInspector) {
   cannotUseCache.push(
     'inspector',
     'internal/util/inspector',

diff --git a/lib/internal/process/coverage.js b/lib/internal/process/coverage.js
@@ -51,12 +51,13 @@ function disableAllAsyncHooks() {
 exports.writeCoverage = writeCoverage;
 
 function setup() {
-  const { Connection } = internalBinding('inspector');
-  if (!Connection) {
+  const { hasInspector } = internalBinding('config');
+  if (!hasInspector) {
     process._rawDebug('inspector not enabled');
     return;
   }
 
+  const { Connection } = internalBinding('inspector');
   coverageConnection = new Connection((res) => {
     if (coverageConnection._coverageCallback) {
       coverageConnection._coverageCallback(res);