lib: make properties on Blob and URL enumerable

nodejs · nodejs-github-bot · Oct 14, 2022 · Oct 8, 2022 · Oct 8, 2022 · Oct 8, 2022
commit 892e453166e27f7f9916fe57f117eacdc78f23b8
diff --git a/lib/internal/blob.js b/lib/internal/blob.js
@@ -4,6 +4,7 @@ const {
   ArrayFrom,
   MathMax,
   MathMin,
+  ObjectDefineProperties,
   ObjectDefineProperty,
   PromiseResolve,
   PromiseReject,
@@ -45,6 +46,7 @@ const {
   createDeferredPromise,
   customInspectSymbol: kInspect,
   kEmptyObject,
+  kEnumerableProperty,
 } = require('internal/util');
 const { inspect } = require('internal/util/inspect');
 
@@ -362,6 +364,15 @@ ObjectDefineProperty(Blob.prototype, SymbolToStringTag, {
   value: 'Blob',
 });
 
+ObjectDefineProperties(Blob.prototype, {
+  size: kEnumerableProperty,
+  type: kEnumerableProperty,
+  slice: kEnumerableProperty,
+  stream: kEnumerableProperty,
+  text: kEnumerableProperty,
+  arrayBuffer: kEnumerableProperty,
+});
+
 function resolveObjectURL(url) {
   url = `${url}`;
   try {

diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -1065,6 +1065,11 @@ ObjectDefineProperties(URL.prototype, {
   toJSON: kEnumerableProperty,
 });
 
+ObjectDefineProperties(URL, {
+  createObjectURL: kEnumerableProperty,
+  revokeObjectURL: kEnumerableProperty,
+});
+
 function update(url, params) {
   if (!url)
     return;

diff --git a/test/parallel/test-blob.js b/test/parallel/test-blob.js
@@ -187,6 +187,23 @@ assert.throws(() => new Blob({}), {
   });
 }
 
+{
+  const descriptors = Object.getOwnPropertyDescriptors(Blob.prototype);
+  const enumerable = [
+    'size',
+    'type',
+    'slice',
+    'stream',
+    'text',
+    'arrayBuffer'
+  ];
+
+  for (const prop of enumerable) {
+    assert.notStrictEqual(descriptors[prop], undefined);
+    assert.strictEqual(descriptors[prop].enumerable, true);
+  }
+}
+
 {
   const b = new Blob(['test', 42]);
   b.text().then(common.mustCall((text) => {

diff --git a/test/parallel/test-whatwg-url-properties.js b/test/parallel/test-whatwg-url-properties.js
@@ -28,6 +28,13 @@ const { URL, URLSearchParams } = require('url');
   testAccessor(URL.prototype, name, readonly);
 });
 
+[
+  { name: 'createObjectURL' },
+  { name: 'revokeObjectURL' },
+].forEach(({ name }) => {
+  testStaticAccessor(URL, name);
+});
+
 [
   { name: 'append' },
   { name: 'delete' },
@@ -98,3 +105,12 @@ function testAccessor(target, name, readonly = false) {
     );
   }
 }
+
+function testStaticAccessor(target, name) {
+  const desc = Object.getOwnPropertyDescriptor(target, name);
+  assert.notDeepStrictEqual(desc, undefined);
+
+  assert.deepStrictEqual(desc.configurable, true);
+  assert.deepStrictEqual(desc.enumerable, true);
+  assert.deepStrictEqual(desc.writable, true);
+}