Convert the RefSet primitive to a proper class and use a Set inte…

…rally

The `RefSet` primitive predates ES6, so that most likely explains why an
object is used internally to track the entries. However, nowadays we can
use built-in JavaScript sets for this purpose. Built-in types are often
more efficient/optimized and using it makes the code a bit more clear
since we don't have to assign `true` to keys anymore just to indicate
their presence.

src/core/primitives.js

@@ -221,28 +221,23 @@ var Ref = (function RefClosure() {
 
 // The reference is identified by number and generation.
 // This structure stores only one instance of the reference.
-var RefSet = (function RefSetClosure() {
-  // eslint-disable-next-line no-shadow
-  function RefSet() {
-    this.dict = Object.create(null);
+class RefSet {
+  constructor() {
+    this._set = new Set();
   }
 
-  RefSet.prototype = {
-    has: function RefSet_has(ref) {
-      return ref.toString() in this.dict;
-    },
-
-    put: function RefSet_put(ref) {
-      this.dict[ref.toString()] = true;
-    },
+  has(ref) {
+    return this._set.has(ref.toString());
+  }
 
-    remove: function RefSet_remove(ref) {
-      delete this.dict[ref.toString()];
-    },
-  };
+  put(ref) {
+    this._set.add(ref.toString());
+  }
 
-  return RefSet;
-})();
+  remove(ref) {
+    this._set.delete(ref.toString());
+  }
+}
 
 var RefSetCache = (function RefSetCacheClosure() {
   // eslint-disable-next-line no-shadow