From ce2f5c2de8180209d8dd3ae0ed39bb59147cc459 Mon Sep 17 00:00:00 2001
From: James M Snell <jasnell@gmail.com>
Date: Tue, 21 Dec 2021 12:05:26 -0800
Subject: [PATCH] buffer: expose Blob as a global

`Blob` is defined as a global in the spec. We have WPT's for it,
and it's graduated experimental. Time to expose it as a global.

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: https://github.com/nodejs/node/pull/41270
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
---
 .eslintrc.js                   |  1 +
 doc/api/buffer.md              |  2 +-
 doc/api/globals.md             | 10 ++++++++++
 lib/.eslintrc.yaml             |  2 ++
 lib/internal/bootstrap/node.js |  7 +++++++
 5 files changed, 21 insertions(+), 1 deletion(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 91a9623c6d905f..890c1663025cf8 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -363,6 +363,7 @@ module.exports = {
     BigInt: 'readable',
     BigInt64Array: 'readable',
     BigUint64Array: 'readable',
+    Blob: 'readable',
     DOMException: 'readable',
     Event: 'readable',
     EventTarget: 'readable',
diff --git a/doc/api/buffer.md b/doc/api/buffer.md
index 9ea44757b12911..80a97ababd2d06 100644
--- a/doc/api/buffer.md
+++ b/doc/api/buffer.md
@@ -458,7 +458,7 @@ added:
   - v14.18.0
 changes:
   - version: REPLACEME
-    pr-url: https://github.com/nodejs/node/pull/00000
+    pr-url: https://github.com/nodejs/node/pull/41270
     description: No longer experimental.
 -->
 
diff --git a/doc/api/globals.md b/doc/api/globals.md
index bb3ca66930e52d..f3a1d46d251300 100644
--- a/doc/api/globals.md
+++ b/doc/api/globals.md
@@ -199,6 +199,16 @@ added: v17.3.0
 
 If `abortSignal.aborted` is `true`, throws `abortSignal.reason`.
 
+## Class: `Blob`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+<!-- type=global -->
+
+See {Blob}.
+
 ## Class: `Buffer`
 
 <!-- YAML
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
index a8881c6b398939..bf8b55a34435b4 100644
--- a/lib/.eslintrc.yaml
+++ b/lib/.eslintrc.yaml
@@ -37,6 +37,8 @@ rules:
       # disabled with --no-harmony-atomics CLI flag.
     - name: Atomics
       message: "Use `const { Atomics } = globalThis;` instead of the global."
+    - name: Blob
+      message: "Use `const { Blob } = require('buffer');` instead of the global."
     - name: Buffer
       message: "Use `const { Buffer } = require('buffer');` instead of the global."
     - name: DOMException
diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
index a6ef700143ee9f..b621a17c0d9e3c 100644
--- a/lib/internal/bootstrap/node.js
+++ b/lib/internal/bootstrap/node.js
@@ -444,6 +444,7 @@ function setupGlobalProxy() {
 
 function setupBuffer() {
   const {
+    Blob,
     Buffer,
     atob,
     btoa,
@@ -456,6 +457,12 @@ function setupBuffer() {
   delete bufferBinding.zeroFill;
 
   ObjectDefineProperties(globalThis, {
+    'Blob': {
+      value: Blob,
+      enumerable: false,
+      writable: true,
+      configurable: true,
+    },
     'Buffer': {
       value: Buffer,
       enumerable: false,