nodejs
diff --git a/Collapse file
‎lib/internal/url.js‎
Copy file name to clipboardExpand all lines: lib/internal/url.js
+2-1Lines changed: 2 additions & 1 deletion b/Collapse file
‎lib/internal/url.js‎
Copy file name to clipboardExpand all lines: lib/internal/url.js
+2-1Lines changed: 2 additions & 1 deletion
diff --git a/Collapse file
‎src/node_url.cc‎
Copy file name to clipboardExpand all lines: src/node_url.cc
+2Lines changed: 2 additions & 0 deletions b/Collapse file
‎src/node_url.cc‎
Copy file name to clipboardExpand all lines: src/node_url.cc
+2Lines changed: 2 additions & 0 deletions
diff --git a/Collapse file
‎src/node_url.h‎
Copy file name to clipboardExpand all lines: src/node_url.h
+2-1Lines changed: 2 additions & 1 deletion b/Collapse file
‎src/node_url.h‎
Copy file name to clipboardExpand all lines: src/node_url.h
+2-1Lines changed: 2 additions & 1 deletion
diff --git a/Collapse file
‎test/fixtures/url-setter-tests.js‎
Copy file name to clipboardExpand all lines: test/fixtures/url-setter-tests.js
+12-1Lines changed: 12 additions & 1 deletion b/Collapse file
‎test/fixtures/url-setter-tests.js‎
Copy file name to clipboardExpand all lines: test/fixtures/url-setter-tests.js
+12-1Lines changed: 12 additions & 1 deletion
@@ -46,6 +46,7 @@ const {
   URL_FLAGS_HAS_PATH,
   URL_FLAGS_HAS_QUERY,
   URL_FLAGS_HAS_USERNAME,
+  URL_FLAGS_IS_DEFAULT_SCHEME_PORT,
   URL_FLAGS_SPECIAL,
   kFragment,
   kHost,
@@ -276,7 +277,7 @@ function onParsePortComplete(flags, protocol, username, password,
 function onParseHostComplete(flags, protocol, username, password,
                              host, port, path, query, fragment) {
   onParseHostnameComplete.apply(this, arguments);
-  if (port !== null)
+  if (port !== null || ((flags & URL_FLAGS_IS_DEFAULT_SCHEME_PORT) !== 0))
     onParsePortComplete.apply(this, arguments);
 }
 
 
@@ -1749,6 +1749,8 @@ void URL::Parse(const char* input,
             }
             // the port is valid
             url->port = NormalizePort(url->scheme, static_cast<int>(port));
+            if (url->port == -1)
+              url->flags |= URL_FLAGS_IS_DEFAULT_SCHEME_PORT;
             buffer.clear();
           } else if (has_state_override) {
             // TODO(TimothyGu): Similar case as above.
 
@@ -50,7 +50,8 @@ using v8::Value;
   XX(URL_FLAGS_HAS_HOST, 0x80)                                                \
   XX(URL_FLAGS_HAS_PATH, 0x100)                                               \
   XX(URL_FLAGS_HAS_QUERY, 0x200)                                              \
-  XX(URL_FLAGS_HAS_FRAGMENT, 0x400)
+  XX(URL_FLAGS_HAS_FRAGMENT, 0x400)                                           \
+  XX(URL_FLAGS_IS_DEFAULT_SCHEME_PORT, 0x800)                                 \
 
 enum url_parse_state {
   kUnknownState = -1,
 
@@ -2,7 +2,7 @@
 
 /* The following tests are copied from WPT. Modifications to them should be
    upstreamed first. Refs:
-   https://github.com/w3c/web-platform-tests/blob/ed4bb727ed/url/setters_tests.json
+   https://github.com/w3c/web-platform-tests/blob/f0fe479/url/setters_tests.json
    License: http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html
 */
 module.exports =
@@ -727,6 +727,17 @@ module.exports =
                 "port": "80"
             }
         },
+        {
+            "comment": "Port number is removed if new port is scheme default and existing URL has a non-default port",
+            "href": "http://example.net:8080",
+            "new_value": "example.com:80",
+            "expected": {
+                "href": "http://example.com/",
+                "host": "example.com",
+                "hostname": "example.com",
+                "port": ""
+            }
+        },
         {
             "comment": "Stuff after a / delimiter is ignored",
             "href": "http://example.net/path",