diff options
Diffstat (limited to 'ext/fetch/22_body.js')
-rw-r--r-- | ext/fetch/22_body.js | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/ext/fetch/22_body.js b/ext/fetch/22_body.js index 49da149c2..acbef4ec4 100644 --- a/ext/fetch/22_body.js +++ b/ext/fetch/22_body.js @@ -372,7 +372,7 @@ return { body, contentType }; } - webidl.converters["BodyInit"] = (V, opts) => { + webidl.converters["BodyInit_DOMString"] = (V, opts) => { // Union for (ReadableStream or Blob or ArrayBufferView or ArrayBuffer or FormData or URLSearchParams or USVString) if (V instanceof ReadableStream) { // TODO(lucacasonato): ReadableStream is not branded @@ -393,10 +393,13 @@ return webidl.converters["ArrayBufferView"](V, opts); } } - return webidl.converters["USVString"](V, opts); + // BodyInit conversion is passed to extractBody(), which calls core.encode(). + // core.encode() will UTF-8 encode strings with replacement, being equivalent to the USV normalization. + // Therefore we can convert to DOMString instead of USVString and avoid a costly redundant conversion. + return webidl.converters["DOMString"](V, opts); }; - webidl.converters["BodyInit?"] = webidl.createNullableConverter( - webidl.converters["BodyInit"], + webidl.converters["BodyInit_DOMString?"] = webidl.createNullableConverter( + webidl.converters["BodyInit_DOMString"], ); window.__bootstrap.fetchBody = { mixinBody, InnerBody, extractBody }; |