Skip to content

Commit

Permalink
url: remove U+0000 case in the fragment state
Browse files Browse the repository at this point in the history
  • Loading branch information
targos committed Sep 8, 2020
1 parent 1dbbf1c commit 3d7899e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions src/node_url.cc
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
Expand Down
3 changes: 0 additions & 3 deletions test/wpt/status/url.json
Original file line number Diff line number Diff line change
Expand Up @@ -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"
},
Expand Down

0 comments on commit 3d7899e

Please sign in to comment.