From 7bc0e6a4e79684a98cb64c61452af00e20f590e7 Mon Sep 17 00:00:00 2001
From: Khafra <maitken033380023@gmail.com>
Date: Wed, 22 Mar 2023 15:28:12 -0400
Subject: [PATCH] buffer: graduate File from experimental and expose as global

PR-URL: https://github.com/nodejs/node/pull/47153
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
---
 doc/api/buffer.md                                      |  6 ++++--
 doc/api/globals.md                                     | 10 ++++++++++
 lib/.eslintrc.yaml                                     |  2 ++
 lib/internal/bootstrap/web/exposed-window-or-worker.js |  2 ++
 lib/internal/file.js                                   |  3 ---
 5 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/doc/api/buffer.md b/doc/api/buffer.md
index d6e2831dd09913..da90d3f3050882 100644
--- a/doc/api/buffer.md
+++ b/doc/api/buffer.md
@@ -5054,10 +5054,12 @@ See [`Buffer.from(string[, encoding])`][`Buffer.from(string)`].
 added:
   - v19.2.0
   - v18.13.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/47153
+    description: No longer experimental.
 -->
 
-> Stability: 1 - Experimental
-
 * Extends: {Blob}
 
 A [`File`][] provides information about files.
diff --git a/doc/api/globals.md b/doc/api/globals.md
index ec42fd53ab24eb..a1b1f259b32418 100644
--- a/doc/api/globals.md
+++ b/doc/api/globals.md
@@ -474,6 +474,16 @@ changes:
 
 A browser-compatible implementation of the [`fetch()`][] function.
 
+## Class: `File`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+<!-- type=global -->
+
+See {File}.
+
 ## Class `FormData`
 
 <!-- YAML
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
index a8716ed7e52b77..cc4fa1975016eb 100644
--- a/lib/.eslintrc.yaml
+++ b/lib/.eslintrc.yaml
@@ -59,6 +59,8 @@ rules:
       message: Use `const { Event } = require('internal/event_target');` instead of the global.
     - name: EventTarget
       message: Use `const { EventTarget } = require('internal/event_target');` instead of the global.
+    - name: File
+      message: Use `const { File } = require('buffer');` instead of the global.
     - name: FormData
       message: Use `const { FormData } = require('internal/deps/undici/undici');` instead of the global.
     - name: Headers
diff --git a/lib/internal/bootstrap/web/exposed-window-or-worker.js b/lib/internal/bootstrap/web/exposed-window-or-worker.js
index ec4671dcd83474..01c88c9679dfc0 100644
--- a/lib/internal/bootstrap/web/exposed-window-or-worker.js
+++ b/lib/internal/bootstrap/web/exposed-window-or-worker.js
@@ -34,6 +34,8 @@ exposeLazyInterfaces(globalThis, 'internal/worker/io', [
 defineLazyProperties(globalThis, 'buffer', ['atob', 'btoa']);
 // https://www.w3.org/TR/FileAPI/#dfn-Blob
 exposeLazyInterfaces(globalThis, 'internal/blob', ['Blob']);
+// https://www.w3.org/TR/FileAPI/#dfn-file
+exposeLazyInterfaces(globalThis, 'internal/file', ['File']);
 // https://www.w3.org/TR/hr-time-2/#the-performance-attribute
 exposeLazyInterfaces(globalThis, 'perf_hooks', [
   'Performance', 'PerformanceEntry', 'PerformanceMark', 'PerformanceMeasure',
diff --git a/lib/internal/file.js b/lib/internal/file.js
index 4c6ca4f61fa4d8..1fc08c45a90f33 100644
--- a/lib/internal/file.js
+++ b/lib/internal/file.js
@@ -13,7 +13,6 @@ const {
 
 const {
   customInspectSymbol: kInspect,
-  emitExperimentalWarning,
   kEnumerableProperty,
   kEmptyObject,
   toUSVString,
@@ -37,8 +36,6 @@ class File extends Blob {
   #lastModified;
 
   constructor(fileBits, fileName, options = kEmptyObject) {
-    emitExperimentalWarning('buffer.File');
-
     if (arguments.length < 2) {
       throw new ERR_MISSING_ARGS('fileBits', 'fileName');
     }