url: fix url spec compliance issues

anonrig · lemire · danielleadams · commit 0c67a7ad573c · 2023-04-11T22:54:52.000-04:00
Co-authored-by: Daniel Lemire <daniel@lemire.me> PR-URL: #46547 Backport-PR-URL: #47435 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -872,6 +872,19 @@ function update(url, params) {
     ctx.search = '?' + serializedParams;
   } else {
     ctx.search = '';
+
+    // Potentially strip trailing spaces from an opaque path
+    if (ctx.hasOpaquePath && ctx.hash.length === 0) {
+      let length = ctx.pathname.length;
+      while (length > 0 && ctx.pathname.charCodeAt(length - 1) === 32) {
+        length--;
+      }
+
+      // No need to copy the whole string if there is no space at the end
+      if (length !== ctx.pathname.length) {
+        ctx.pathname = ctx.pathname.slice(0, length);
+      }
+    }
   }
   ctx.href = constructHref(ctx);
 }
diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
@@ -7,7 +7,13 @@
     "skip": "TODO: port from .window.js"
   },
   "historical.any.js": {
-    "requires": ["small-icu"]
+    "requires": ["small-icu"],
+    "fail": {
+      "expected": [
+        "URL: no structured serialize/deserialize support",
+        "URLSearchParams: no structured serialize/deserialize support"
+      ]
+    }
   },
   "urlencoded-parser.any.js": {
     "requires": ["small-icu"]
diff --git a/test/wpt/test-url.js b/test/wpt/test-url.js
@@ -15,6 +15,6 @@ runner.setScriptModifier((obj) => {
 });
 runner.pretendGlobalThisAs('Window');
 runner.setInitScript(`
-  globalThis.location = {};
+  globalThis.location ||= {};
 `);
 runner.runJsTests();
