url: remove U+0000 case in the fragment state

Port of whatwg/url#486 PR-URL: #33770 Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Shingo Inoue <leko.noor@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
nodejs · Sep 16, 2020 · 4831278 · 4831278
1 parent eb9d7a4
commit 4831278
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/src/node_url.cc b/src/node_url.cc
@@ -2084,8 +2084,6 @@ void URL::Parse(const char* input,
             url->flags |= URL_FLAGS_HAS_FRAGMENT;
             url->fragment = std::move(buffer);
             break;
-          case 0:
-            break;
           default:
             AppendOrEscape(&buffer, ch, FRAGMENT_ENCODE_SET);
         }

diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
@@ -12,9 +12,6 @@
   "idlharness.any.js": {
     "fail": "getter/setter names are wrong, etc."
   },
-  "url-setters-stripping.any.js": {
-    "fail": "TODO: port https://github.com/whatwg/url/pull/486"
-  },
   "urlsearchparams-constructor.any.js": {
     "fail": "TODO: fix https://github.com/nodejs/node/issues/33892"
   },