perf(webidl): inline ResponseInit converter (#12285)

2 files changed, 46 insertions, 3 deletions

diff --git a/cli/tests/unit/response_test.ts b/cli/tests/unit/response_test.ts
index 3f6197e1f..9c6da1c5f 100644
--- a/cli/tests/unit/response_test.ts
+++ b/cli/tests/unit/response_test.ts
@@ -3,6 +3,7 @@ import {
   assert,
   assertEquals,
   assertStringIncludes,
+  assertThrows,
   unitTest,
 } from "./test_util.ts";
 
@@ -56,6 +57,23 @@ unitTest(async function responseFormData() {
   assertEquals([...formData], [...input]);
 });
 
+unitTest(function responseInvalidInit() {
+  // deno-lint-ignore ban-ts-comment
+  // @ts-expect-error
+  assertThrows(() => new Response("", 0));
+  assertThrows(() => new Response("", { status: 0 }));
+  // deno-lint-ignore ban-ts-comment
+  // @ts-expect-error
+  assertThrows(() => new Response("", { status: null }));
+});
+
+unitTest(function responseNullInit() {
+  // deno-lint-ignore ban-ts-comment
+  // @ts-expect-error
+  const response = new Response("", null);
+  assertEquals(response.status, 200);
+});
+
 unitTest(function customInspectFunction() {
   const response = new Response();
   assertEquals(
diff --git a/ext/fetch/23_response.js b/ext/fetch/23_response.js
index 29a37ceca..5ff6e56c3 100644
--- a/ext/fetch/23_response.js
+++ b/ext/fetch/23_response.js
@@ -12,6 +12,7 @@
 "use strict";
 
 ((window) => {
+  const { isProxy } = Deno.core;
   const webidl = window.__bootstrap.webidl;
   const consoleInternal = window.__bootstrap.console;
   const { HTTP_TAB_OR_SPACE, regexMatcher } = window.__bootstrap.infra;
@@ -249,7 +250,7 @@
         prefix,
         context: "Argument 1",
       });
-      init = webidl.converters["ResponseInit"](init, {
+      init = webidl.converters["ResponseInit_fast"](init, {
         prefix,
         context: "Argument 2",
       });
@@ -260,7 +261,10 @@
         );
       }
 
-      if (!RegExpPrototypeTest(REASON_PHRASE_RE, init.statusText)) {
+      if (
+        init.statusText &&
+        !RegExpPrototypeTest(REASON_PHRASE_RE, init.statusText)
+      ) {
         throw new TypeError("Status text is not valid.");
       }
 
@@ -270,7 +274,7 @@
       this[_response] = response;
       /** @type {Headers} */
       this[_headers] = headersFromHeaderList(response.headerList, "response");
-      if (init.headers !== undefined) {
+      if (init.headers) {
         fillHeaders(this[_headers], init.headers);
       }
       if (body !== null) {
@@ -407,6 +411,27 @@
       converter: webidl.converters["HeadersInit"],
     }],
   );
+  webidl.converters["ResponseInit_fast"] = function (init, opts) {
+    if (init === undefined || init === null) {
+      return { status: 200, statusText: "", headers: undefined };
+    }
+    // Fast path, if not a proxy
+    if (typeof init === "object" && !isProxy(init)) {
+      // Not a proxy fast path
+      const status = init.status !== undefined
+        ? webidl.converters["unsigned short"](init.status)
+        : 200;
+      const statusText = init.statusText !== undefined
+        ? webidl.converters["ByteString"](init.statusText)
+        : "";
+      const headers = init.headers !== undefined
+        ? webidl.converters["HeadersInit"](init.headers)
+        : undefined;
+      return { status, statusText, headers };
+    }
+    // Slow default path
+    return webidl.converters["ResponseInit"](init, opts);
+  };
 
   /**
    * @param {Response} response