buffer: add @@toStringTag to Blob

cjihrig · danielleadams · commit c160d88c9e42 · 2021-02-16T09:16:18.000-05:00
This commit adds the toStringTag to the Blob class to match the behavior of Chrome and Firefox. PR-URL: #37336 Fixes: #37337 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/internal/blob.js b/lib/internal/blob.js
@@ -2,12 +2,14 @@
 
 const {
   ArrayFrom,
+  ObjectDefineProperty,
   ObjectSetPrototypeOf,
   PromiseResolve,
   RegExpPrototypeTest,
   StringPrototypeToLowerCase,
   Symbol,
   SymbolIterator,
+  SymbolToStringTag,
   Uint8Array,
 } = primordials;
 
@@ -217,6 +219,11 @@ class Blob extends JSTransferable {
   }
 }
 
+ObjectDefineProperty(Blob.prototype, SymbolToStringTag, {
+  configurable: true,
+  value: 'Blob',
+});
+
 InternalBlob.prototype.constructor = Blob;
 ObjectSetPrototypeOf(
   InternalBlob.prototype,
diff --git a/test/parallel/test-blob.js b/test/parallel/test-blob.js
@@ -184,3 +184,14 @@ assert.throws(() => new Blob(['test', 1]), {
   const b = new Blob(['hello'], { type: '\x01' });
   assert.strictEqual(b.type, '');
 }
+
+{
+  const descriptor =
+      Object.getOwnPropertyDescriptor(Blob.prototype, Symbol.toStringTag);
+  assert.deepStrictEqual(descriptor, {
+    configurable: true,
+    enumerable: false,
+    value: 'Blob',
+    writable: false
+  });
+}
