Address review comments

Co-authored-by: Timothy Gu <[email protected]>

Author: ExE-Boss

README.md

@@ -479,7 +479,6 @@ Notable missing features include:
 - `[AllowShared]`
 - `[Default]` (for `toJSON()` operations)
 - `[Global]`'s various consequences, including the named properties object and `[[SetPrototypeOf]]`
-- `[Exposed]`
 - `[LenientSetter]`
 - `[LenientThis]`
 - `[NamedConstructor]`

lib/constructs/callback-interface.js

@@ -199,15 +199,7 @@ class CallbackInterface {
   generateInstall() {
     if (this.constants.size > 0) {
       this.str += `
-        const exposed = new Set([
-      `;
-
-      for (const globalName of this.exposed) {
-        this.str += `"${globalName}",\n`;
-      }
-
-      this.str += `
-        ]);
+        const exposed = new Set(${JSON.stringify([...this.exposed])});
       `;
     }
 

lib/constructs/interface.js

@@ -1463,15 +1463,7 @@ class Interface {
     const { idl, name } = this;
 
     this.str += `
-      const exposed = new Set([
-    `;
-
-    for (const globalName of this.exposed) {
-      this.str += `"${globalName}",\n`;
-    }
-
-    this.str += `
-      ]);
+      const exposed = new Set(${JSON.stringify([...this.exposed])});
 
       exports.install = (globalObject, globalNames) => {
         let isExposed = false;