lib: remove unnecessary require

BridgeAR · MylesBorins · commit bd1319397901 · 2018-05-22T14:13:28.000-04:00
The call to url was not necessary since the `url` module itself was already taking care of this. PR-URL: #20567 Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
@@ -89,10 +89,6 @@
       setupGlobalURL();
     }
 
-    // Ensure setURLConstructor() is called before the native
-    // URL::ToObject() method is used.
-    NativeModule.require('internal/url');
-
     // On OpenBSD process.execPath will be relative unless we
     // get the full path before process.execPath is used.
     if (process.platform === 'openbsd') {
diff --git a/lib/url.js b/lib/url.js
@@ -30,6 +30,8 @@ const {
   ERR_INVALID_ARG_TYPE
 } = require('internal/errors').codes;
 
+// This ensures setURLConstructor() is called before the native
+// URL::ToObject() method is used.
 const { spliceOne } = require('internal/util');
 
 // WHATWG URL implementation provided by internal/url
