lib: distinguish webidl interfaces with the extended property "Exposed"

PR-URL: nodejs#46809
Refs: nodejs#42528
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Author: legendecas

lib/internal/bootstrap/browser.js lib/internal/bootstrap/web/exposed-wildcard.js

@@ -1,17 +1,21 @@
 'use strict';
 
+/**
+ * This file exposes web interfaces that is defined with the WebIDL
+ * [Exposed=*] extended attribute.
+ * See more details at https://webidl.spec.whatwg.org/#Exposed.
+ */
+
 const {
-  ObjectDefineProperty,
   globalThis,
 } = primordials;
 
 const {
-  defineOperation,
   exposeInterface,
   lazyDOMExceptionClass,
-  defineLazyProperties,
-  defineReplaceableLazyAttribute,
   exposeLazyInterfaces,
+  exposeGetterAndSetter,
+  exposeNamespace,
 } = require('internal/util');
 const config = internalBinding('config');
 
@@ -35,36 +39,17 @@ exposeGetterAndSetter(globalThis,
                         exposeInterface(globalThis, 'DOMException', value);
                       });
 
-// https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalscope
-const timers = require('timers');
-defineOperation(globalThis, 'clearInterval', timers.clearInterval);
-defineOperation(globalThis, 'clearTimeout', timers.clearTimeout);
-defineOperation(globalThis, 'setInterval', timers.setInterval);
-defineOperation(globalThis, 'setTimeout', timers.setTimeout);
-
+// https://dom.spec.whatwg.org/#interface-abortcontroller
 // Lazy ones.
-exposeLazyInterfaces(globalThis, 'internal/worker/io', ['BroadcastChannel']);
 exposeLazyInterfaces(globalThis, 'internal/abort_controller', [
   'AbortController', 'AbortSignal',
 ]);
+// https://dom.spec.whatwg.org/#interface-eventtarget
 const {
   EventTarget, Event,
 } = require('internal/event_target');
 exposeInterface(globalThis, 'Event', Event);
 exposeInterface(globalThis, 'EventTarget', EventTarget);
-exposeLazyInterfaces(globalThis, 'internal/worker/io', [
-  'MessageChannel', 'MessagePort', 'MessageEvent',
-]);
-defineLazyProperties(globalThis, 'buffer', ['atob', 'btoa']);
-// https://www.w3.org/TR/FileAPI/#dfn-Blob
-exposeLazyInterfaces(globalThis, 'internal/blob', ['Blob']);
-// https://www.w3.org/TR/hr-time-2/#the-performance-attribute
-exposeLazyInterfaces(globalThis, 'perf_hooks', [
-  'Performance', 'PerformanceEntry', 'PerformanceMark', 'PerformanceMeasure',
-  'PerformanceObserver', 'PerformanceObserverEntryList', 'PerformanceResourceTiming',
-]);
-
-defineReplaceableLazyAttribute(globalThis, 'perf_hooks', ['performance']);
 
 // https://encoding.spec.whatwg.org/#textencoder
 // https://encoding.spec.whatwg.org/#textdecoder
@@ -87,27 +72,6 @@ function createGlobalConsole() {
   return consoleFromNode;
 }
 
-// https://heycam.github.io/webidl/#es-namespaces
-function exposeNamespace(target, name, namespaceObject) {
-  ObjectDefineProperty(target, name, {
-    __proto__: null,
-    writable: true,
-    enumerable: false,
-    configurable: true,
-    value: namespaceObject,
-  });
-}
-
-function exposeGetterAndSetter(target, name, getter, setter = undefined) {
-  ObjectDefineProperty(target, name, {
-    __proto__: null,
-    enumerable: false,
-    configurable: true,
-    get: getter,
-    set: setter,
-  });
-}
-
 // Web Streams API
 exposeLazyInterfaces(
   globalThis,

lib/internal/bootstrap/web/exposed-window-or-worker.js

@@ -0,0 +1,47 @@
+'use strict';
+
+/**
+ * This file exposes web interfaces that is defined with the WebIDL
+ * Exposed=(Window,Worker) extended attribute or exposed in
+ * WindowOrWorkerGlobalScope mixin.
+ * See more details at https://webidl.spec.whatwg.org/#Exposed and
+ * https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalscope.
+ */
+
+const {
+  globalThis,
+} = primordials;
+
+const {
+  defineOperation,
+  defineLazyProperties,
+  defineReplaceableLazyAttribute,
+  exposeLazyInterfaces,
+} = require('internal/util');
+
+// https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalscope
+const timers = require('timers');
+defineOperation(globalThis, 'clearInterval', timers.clearInterval);
+defineOperation(globalThis, 'clearTimeout', timers.clearTimeout);
+defineOperation(globalThis, 'setInterval', timers.setInterval);
+defineOperation(globalThis, 'setTimeout', timers.setTimeout);
+
+// https://html.spec.whatwg.org/multipage/web-messaging.html#broadcasting-to-other-browsing-contexts
+exposeLazyInterfaces(globalThis, 'internal/worker/io', ['BroadcastChannel']);
+exposeLazyInterfaces(globalThis, 'internal/worker/io', [
+  'MessageChannel', 'MessagePort', 'MessageEvent',
+]);
+defineLazyProperties(globalThis, 'buffer', ['atob', 'btoa']);
+// https://www.w3.org/TR/FileAPI/#dfn-Blob
+exposeLazyInterfaces(globalThis, 'internal/blob', ['Blob']);
+// https://www.w3.org/TR/hr-time-2/#the-performance-attribute
+exposeLazyInterfaces(globalThis, 'perf_hooks', [
+  'Performance', 'PerformanceEntry', 'PerformanceMark', 'PerformanceMeasure',
+  'PerformanceObserver', 'PerformanceObserverEntryList', 'PerformanceResourceTiming',
+]);
+
+defineReplaceableLazyAttribute(globalThis, 'perf_hooks', ['performance']);
+
+// https://w3c.github.io/FileAPI/#creating-revoking
+const { installObjectURLMethods } = require('internal/url');
+installObjectURLMethods();

lib/internal/process/pre_execution.js

@@ -262,7 +262,7 @@ function setupFetch() {
   });
 }
 
-// TODO(aduh95): move this to internal/bootstrap/browser when the CLI flag is
+// TODO(aduh95): move this to internal/bootstrap/web/* when the CLI flag is
 //               removed.
 function setupWebCrypto() {
   if (process.config.variables.node_no_browser_globals ||
@@ -310,7 +310,7 @@ function setupCodeCoverage() {
   }
 }
 
-// TODO(daeyeon): move this to internal/bootstrap/browser when the CLI flag is
+// TODO(daeyeon): move this to internal/bootstrap/web/* when the CLI flag is
 //                removed.
 function setupCustomEvent() {
   if (process.config.variables.node_no_browser_globals ||

lib/internal/url.js

@@ -93,11 +93,6 @@ const {
   updateUrl,
 } = internalBinding('url');
 
-const {
-  storeDataObject,
-  revokeDataObject,
-} = internalBinding('blob');
-
 const FORWARD_SLASH = /\//g;
 
 const context = Symbol('context');
@@ -782,35 +777,6 @@ class URL {
     return this[context].href;
   }
 
-  static createObjectURL(obj) {
-    const cryptoRandom = lazyCryptoRandom();
-    if (cryptoRandom === undefined)
-      throw new ERR_NO_CRYPTO();
-
-    const blob = lazyBlob();
-    if (!blob.isBlob(obj))
-      throw new ERR_INVALID_ARG_TYPE('obj', 'Blob', obj);
-
-    const id = cryptoRandom.randomUUID();
-
-    storeDataObject(id, obj[blob.kHandle], obj.size, obj.type);
-
-    return `blob:nodedata:${id}`;
-  }
-
-  static revokeObjectURL(url) {
-    url = `${url}`;
-    try {
-      // TODO(@anonrig): Remove this try/catch by calling `parse` directly.
-      const parsed = new URL(url);
-      const split = StringPrototypeSplit(parsed.pathname, ':');
-      if (split.length === 2)
-        revokeDataObject(split[1]);
-    } catch {
-      // If there's an error, it's ignored.
-    }
-  }
-
   static canParse(url, base = undefined) {
     url = `${url}`;
 
@@ -841,8 +807,6 @@ ObjectDefineProperties(URL.prototype, {
 });
 
 ObjectDefineProperties(URL, {
-  createObjectURL: kEnumerableProperty,
-  revokeObjectURL: kEnumerableProperty,
   canParse: {
     __proto__: null,
     configurable: true,
@@ -851,6 +815,59 @@ ObjectDefineProperties(URL, {
   },
 });
 
+function installObjectURLMethods() {
+  const {
+    storeDataObject,
+    revokeDataObject,
+  } = internalBinding('blob');
+
+  function createObjectURL(obj) {
+    const cryptoRandom = lazyCryptoRandom();
+    if (cryptoRandom === undefined)
+      throw new ERR_NO_CRYPTO();
+
+    const blob = lazyBlob();
+    if (!blob.isBlob(obj))
+      throw new ERR_INVALID_ARG_TYPE('obj', 'Blob', obj);
+
+    const id = cryptoRandom.randomUUID();
+
+    storeDataObject(id, obj[blob.kHandle], obj.size, obj.type);
+
+    return `blob:nodedata:${id}`;
+  }
+
+  function revokeObjectURL(url) {
+    url = `${url}`;
+    try {
+      // TODO(@anonrig): Remove this try/catch by calling `parse` directly.
+      const parsed = new URL(url);
+      const split = StringPrototypeSplit(parsed.pathname, ':');
+      if (split.length === 2)
+        revokeDataObject(split[1]);
+    } catch {
+      // If there's an error, it's ignored.
+    }
+  }
+
+  ObjectDefineProperties(URL, {
+    createObjectURL: {
+      __proto__: null,
+      configurable: true,
+      writable: true,
+      enumerable: true,
+      value: createObjectURL,
+    },
+    revokeObjectURL: {
+      __proto__: null,
+      configurable: true,
+      writable: true,
+      enumerable: true,
+      value: revokeObjectURL,
+    },
+  });
+}
+
 // application/x-www-form-urlencoded parser
 // Ref: https://url.spec.whatwg.org/#concept-urlencoded-parser
 function parseParams(qs) {
@@ -1341,6 +1358,7 @@ module.exports = {
   pathToFileURL,
   toPathIfFileURL,
   isURLInstance,
+  installObjectURLMethods,
   URL,
   URLSearchParams,
   domainToASCII,

lib/internal/util.js

@@ -511,6 +511,27 @@ function exposeInterface(target, name, interfaceObject) {
   });
 }
 
+// https://heycam.github.io/webidl/#es-namespaces
+function exposeNamespace(target, name, namespaceObject) {
+  ObjectDefineProperty(target, name, {
+    __proto__: null,
+    writable: true,
+    enumerable: false,
+    configurable: true,
+    value: namespaceObject,
+  });
+}
+
+function exposeGetterAndSetter(target, name, getter, setter = undefined) {
+  ObjectDefineProperty(target, name, {
+    __proto__: null,
+    enumerable: false,
+    configurable: true,
+    get: getter,
+    set: setter,
+  });
+}
+
 function defineLazyProperties(target, id, keys, enumerable = true) {
   const descriptors = { __proto__: null };
   let mod;
@@ -750,6 +771,8 @@ module.exports = {
   emitExperimentalWarning,
   exposeInterface,
   exposeLazyInterfaces,
+  exposeNamespace,
+  exposeGetterAndSetter,
   filterDuplicateStrings,
   filterOwnProperties,
   getConstructorOf,

src/node_realm.cc

@@ -218,9 +218,10 @@ MaybeLocal<Value> Realm::BootstrapNode() {
   }
 
   if (!env_->no_browser_globals()) {
-    result = ExecuteBootstrapper("internal/bootstrap/browser");
-
-    if (result.IsEmpty()) {
+    if (ExecuteBootstrapper("internal/bootstrap/web/exposed-wildcard")
+            .IsEmpty() ||
+        ExecuteBootstrapper("internal/bootstrap/web/exposed-window-or-worker")
+            .IsEmpty()) {
       return MaybeLocal<Value>();
     }
   }