lib: replace WeakMap global by the primordials

Sebastien-Ahkrin · Jan 2, 2020 · Jan 4, 2020 · 7527c467bcd758c3ebc7451e6c0883c7516067ab
commit 7527c467bcd758c3ebc7451e6c0883c7516067ab
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -29,6 +29,8 @@ rules:
       message: "Use `const { Reflect } = primordials;` instead of the global."
     - name: Symbol
       message: "Use `const { Symbol } = primordials;` instead of the global."
+    - name: WeakMap
+      message: "Use `const { WeakMap } = primordials;` instead of the global."
   no-restricted-syntax:
     # Config copied from .eslintrc.js
     - error

diff --git a/lib/internal/console/constructor.js b/lib/internal/console/constructor.js
@@ -17,6 +17,7 @@ const {
   ReflectOwnKeys,
   Symbol,
   SymbolHasInstance,
+  WeakMap,
 } = primordials;
 
 const { trace } = internalBinding('trace_events');

diff --git a/lib/internal/errors.js b/lib/internal/errors.js
@@ -18,6 +18,7 @@ const {
   ObjectKeys,
   Symbol,
   SymbolFor,
+  WeakMap,
 } = primordials;
 
 const messages = new Map();

diff --git a/lib/internal/process/promises.js b/lib/internal/process/promises.js
@@ -2,6 +2,7 @@
 
 const {
   ObjectDefineProperty,
+  WeakMap,
 } = primordials;
 
 const {

diff --git a/lib/internal/source_map/source_map_cache.js b/lib/internal/source_map/source_map_cache.js
@@ -7,6 +7,7 @@ const {
   ObjectGetOwnPropertyDescriptor,
   ObjectPrototypeHasOwnProperty,
   MapPrototypeEntries,
+  WeakMap,
   WeakMapPrototypeGet,
   uncurryThis,
 } = primordials;

diff --git a/lib/internal/vm/module.js b/lib/internal/vm/module.js
@@ -7,6 +7,7 @@ const {
   ObjectDefineProperty,
   SafePromise,
   Symbol,
+  WeakMap,
 } = primordials;
 
 const { isContext } = internalBinding('contextify');
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,7 @@ @@
     const {
       ObjectDefineProperty,
+      WeakMap,
     } = primordials;
     const {
@@ Expand Down @@