lib: add FormData global when fetch is enabled

PR-URL: #41956 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>

Author: targos

.eslintrc.js

@@ -319,6 +319,7 @@ module.exports ={
 Crypto: 'readable',
 CryptoKey: 'readable',
 fetch: 'readable',
+FormData: 'readable',
 globalThis: 'readable',
 Response: 'readable',
 SubtleCrypto: 'readable',

doc/api/globals.md

@@ -363,6 +363,17 @@ added: v17.5.0
 
 A browser-compatible implementation of the [`fetch()`][] function.
 
+## Class `FormData`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental. Enable this API with the [`--experimental-fetch`][]
+> CLI flag.
+
+A browser-compatible implementation of{FormData}.
+
 ## `global`
 
 <!-- YAML

lib/.eslintrc.yaml

@@ -47,6 +47,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: FormData
+message: Use `const{FormData } = require('internal/deps/undici/undici');` instead of the global.
  - name: Headers
 message: Use `const{Headers } = require('internal/deps/undici/undici');` instead of the global.
 # Intl is not available in primordials because it can be

lib/internal/bootstrap/pre_execution.js

@@ -163,6 +163,12 @@ function setupFetch(){
 configurable: true,
 value: undici.fetch
 });
+ObjectDefineProperty(globalThis,'FormData',{
+writable: true,
+enumerable: false,
+configurable: true,
+value: undici.FormData
+});
 ObjectDefineProperty(globalThis,'Headers',{
 writable: true,
 enumerable: false,

test/common/index.js

@@ -303,6 +303,7 @@ if (global.structuredClone){
 if(global.fetch){
 knownGlobals.push(
 global.fetch,
+global.FormData,
 global.Request,
 global.Response,
 global.Headers,

test/parallel/test-fetch.mjs

@@ -7,6 +7,7 @@ import events from 'events'
 importhttpfrom'http';
 
 assert.strictEqual(typeofglobalThis.fetch,'function');
+assert.strictEqual(typeofglobalThis.FormData,'function');
 assert.strictEqual(typeofglobalThis.Headers,'function');
 assert.strictEqual(typeofglobalThis.Request,'function');
 assert.strictEqual(typeofglobalThis.Response,'function');

test/wpt/test-url.js

@@ -11,7 +11,7 @@ runner.setScriptModifier((obj) =>{
 // created via `document.createElement`. So we need to ignore them and just
 // test `URL`.
 obj.code=obj.code.replace(/\["url","a","area"\]/,'[ "url" ]');
-}elseif(typeofFormData==='undefined'&&// eslint-disable-line
+}elseif(typeofFormData==='undefined'&&
 obj.filename.includes('urlsearchparams-constructor.any.js')){
 // TODO(XadillaX): Remove this `else if` after `FormData` is supported.
 

tools/doc/type-parser.mjs

@@ -271,6 +271,7 @@ const customTypesMap ={
 'TextDecoderStream':
 'webstreams.md#class-textdecoderstream',
 
+'FormData': 'https://developer.mozilla.org/en-US/docs/Web/API/FormData',
 'Headers': 'https://developer.mozilla.org/en-US/docs/Web/API/Headers',
 'Response': 'https://developer.mozilla.org/en-US/docs/Web/API/Response',
 'Request': 'https://developer.mozilla.org/en-US/docs/Web/API/Request',