Make hostname state return early if a port is given

Align with whatwg/url#604.
jsdom · Jun 8, 2021 · f12bc66 · f12bc66
1 parent 058fa4f
commit f12bc66
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/scripts/get-latest-platform-tests.js b/scripts/get-latest-platform-tests.js
@@ -24,7 +24,7 @@ process.on("unhandledRejection", err => {
 // 1. Go to https://github.com/w3c/web-platform-tests/tree/master/url
 // 2. Press "y" on your keyboard to get a permalink
 // 3. Copy the commit hash
-const commitHash = "b1fc334f29f80f3160e98a1fc479b0d44b54ffd8";
+const commitHash = "67a580b4a11da1dca6c1195a2b670e361e4013c9";
 
 const urlPrefix = `https://raw.githubusercontent.com/web-platform-tests/wpt/${commitHash}/url/`;
 const targetDir = path.resolve(__dirname, "..", "test", "web-platform-tests");

diff --git a/src/url-state-machine.js b/src/url-state-machine.js
@@ -760,6 +760,10 @@ URLStateMachine.prototype["parse host"] = function parseHostName(c, cStr) {
       return failure;
     }
 
+    if (this.stateOverride === "hostname") {
+      return false;
+    }
+
     const host = parseHost(this.buffer, isNotSpecial(this.url));
     if (host === failure) {
       return failure;
@@ -768,9 +772,6 @@ URLStateMachine.prototype["parse host"] = function parseHostName(c, cStr) {
     this.url.host = host;
     this.buffer = "";
     this.state = "port";
-    if (this.stateOverride === "hostname") {
-      return false;
-    }
   } else if (isNaN(c) || c === p("/") || c === p("?") || c === p("#") ||
              (isSpecial(this.url) && c === p("\\"))) {
     --this.pointer;