Remove hideInternalModules API

lib/browser/init.js

@@ -18,10 +18,6 @@ require('../common/init')
 
 var globalPaths = Module.globalPaths
 
-if (!process.env.ELECTRON_HIDE_INTERNAL_MODULES) {
-  globalPaths.push(path.join(__dirname, 'api'))
-}
-
 // Expose public APIs.
 globalPaths.push(path.join(__dirname, 'api', 'exports'))
 

lib/common/api/exports/electron.js

@@ -1,20 +1,6 @@
-// Do not expose the internal modules to `require`.
-const hideInternalModules = function () {
-  var globalPaths = require('module').globalPaths
-  if (globalPaths.length === 3) {
-    // Remove the "common/api/lib" and "browser-or-renderer/api/lib".
-    return globalPaths.splice(0, 2)
-  }
-}
-
 // Attaches properties to |exports|.
 exports.defineProperties = function (exports) {
   return Object.defineProperties(exports, {
-    hideInternalModules: {
-      enumerable: true,
-      value: hideInternalModules
-    },
-
     // Common modules, please sort with alphabet order.
     clipboard: {
       // Must be enumerable, otherwise it woulde be invisible to remote module.

lib/common/init.js

@@ -12,11 +12,6 @@ process.atomBinding = function (name) {
   }
 }
 
-if (!process.env.ELECTRON_HIDE_INTERNAL_MODULES) {
-  // Add common/api/lib to module search paths.
-  Module.globalPaths.push(path.join(__dirname, 'api'))
-}
-
 // setImmediate and process.nextTick makes use of uv_check and uv_prepare to
 // run the callbacks, however since we only run uv loop on requests, the
 // callbacks wouldn't be called until something else activated the uv loop,

lib/renderer/init.js

@@ -16,10 +16,6 @@ require('../common/init')
 
 var globalPaths = Module.globalPaths
 
-if (!process.env.ELECTRON_HIDE_INTERNAL_MODULES) {
-  globalPaths.push(path.join(__dirname, 'api'))
-}
-
 // Expose public APIs.
 globalPaths.push(path.join(__dirname, 'api', 'exports'))