wpt: add scheme-about.any.js

Author: KhafraDev

lib/fetch/index.js

@@ -753,31 +753,17 @@ async function schemeFetch (fetchParams) {
   // let request be fetchParams’s request
   const { request } = fetchParams
 
-  const {
-    protocol: scheme,
-    pathname: path
-  } = requestCurrentURL(request)
+  const { protocol: scheme } = requestCurrentURL(request)
 
   // switch on request’s current URL’s scheme, and run the associated steps:
   switch (scheme) {
     case 'about:': {
       // If request’s current URL’s path is the string "blank", then return a new response
       // whose status message is `OK`, header list is « (`Content-Type`, `text/html;charset=utf-8`) »,
       // and body is the empty byte sequence.
-      if (path === 'blank') {
-        const resp = makeResponse({
-          statusText: 'OK',
-          headersList: [
-            ['content-type', 'text/html;charset=utf-8']
-          ]
-        })
-
-        resp.urlList = [new URL('about:blank')]
-        return resp
-      }
 
       // Otherwise, return a network error.
-      return makeNetworkError('invalid path called')
+      return makeNetworkError('about scheme is not supported')
     }
     case 'blob:': {
       resolveObjectURL = resolveObjectURL || require('buffer').resolveObjectURL

lib/fetch/response.js

@@ -5,7 +5,6 @@ const { extractBody, cloneBody, mixinBody } = require('./body')
 const util = require('../core/util')
 const { kEnumerableProperty } = util
 const {
-  responseURL,
   isValidReasonPhrase,
   isCancelled,
   isAborted,

test/wpt/tests/fetch/api/basic/scheme-about.any.js

@@ -0,0 +1,26 @@
+// META: global=window,worker
+// META: script=../resources/utils.js
+
+function checkNetworkError(url, method) {
+  method = method || "GET";
+  const desc = "Fetching " + url.substring(0, 45) + " with method " + method + " is KO"
+  promise_test(function(test) {
+    var promise = fetch(url, { method: method });
+    return promise_rejects_js(test, TypeError, promise);
+  }, desc);
+}
+
+checkNetworkError("about:blank", "GET");
+checkNetworkError("about:blank", "PUT");
+checkNetworkError("about:blank", "POST");
+checkNetworkError("about:invalid.com");
+checkNetworkError("about:config");
+checkNetworkError("about:unicorn");
+
+promise_test(function(test) {
+  var promise = fetch("about:blank", {
+    "method": "GET",
+    "Range": "bytes=1-10"
+  });
+  return promise_rejects_js(test, TypeError, promise);
+}, "Fetching about:blank with range header does not affect behavior");