url: conform to origin getter spec changes

PR-URL: nodejs#48319 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
SaxenaShiv · Jun 7, 2023 · f2670a6 · f2670a6
1 parent e6e0110
commit f2670a6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -840,7 +840,9 @@ class URL {
       if (path.length > 0) {
         try {
           const out = new URL(path);
-          if (out.#context.scheme_type !== 1) {
+          // Only return origin of scheme is `http` or `https`
+          // Otherwise return a new opaque origin (null).
+          if (out.#context.scheme_type === 0 || out.#context.scheme_type === 2) {
             return `${out.protocol}//${out.host}`;
           }
         } catch {

diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
@@ -17,8 +17,5 @@
   },
   "javascript-urls.window.js": {
     "skip": "requires document.body reference"
-  },
-  "url-origin.any.js": {
-    "skip": "implementation not up to date"
   }
 }