url: conform to origin getter spec changes

PR-URL: #48319 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
nodejs · Aug 31, 2023 · f59c963 · f59c963
1 parent cb92c4b
commit f59c963
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
@@ -731,7 +731,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
@@ -15,8 +15,5 @@
   },
   "javascript-urls.window.js": {
     "skip": "requires document.body reference"
-  },
-  "url-origin.any.js": {
-    "skip": "implementation not up to date"
   }
 }