url: percent-encode NULL bytes in fragments

Per the change in whatwg/url#486.

Author: djc

src/parser.rs

@@ -1480,9 +1480,9 @@ impl<'a> Parser<'a> {
                 self.log_violation(SyntaxViolation::NullInFragment)
             } else {
                 self.check_url_code_point(c, &input);
-                self.serialization
-                    .extend(utf8_percent_encode(utf8_c, FRAGMENT));
             }
+            self.serialization
+                .extend(utf8_percent_encode(utf8_c, FRAGMENT));
         }
     }
 

tests/setters_tests.json

@@ -1800,15 +1800,6 @@
                 "hash": "#foo%60bar"
             }
         },
-        {
-            "comment": "Simple percent-encoding; nuls, tabs, and newlines are removed",
-            "href": "a:/",
-            "new_value": "\u0000\u0001\t\n\r\u001f !\"#$%&'()*+,-./09:;<=>?@AZ[\\]^_`az{|}~\u007f\u0080\u0081Éé",
-            "expected": {
-                "href": "a:/#%01%1F%20!%22#$%&'()*+,-./09:;%3C=%3E?@AZ[\\]^_%60az{|}~%7F%C2%80%C2%81%C3%89%C3%A9",
-                "hash": "#%01%1F%20!%22#$%&'()*+,-./09:;%3C=%3E?@AZ[\\]^_%60az{|}~%7F%C2%80%C2%81%C3%89%C3%A9"
-            }
-        },
         {
             "comment": "Bytes already percent-encoded are left as-is",
             "href": "http://example.net",

tests/urltestdata.json

@@ -6562,20 +6562,5 @@
     "pathname": "/test-a-colon-slash-slash-b.html",
     "search": "",
     "hash": ""
-  },
-  "Null code point in fragment",
-  {
-    "input": "http://example.org/test?a#b\u0000c",
-    "base": "about:blank",
-    "href": "http://example.org/test?a#bc",
-    "protocol": "http:",
-    "username": "",
-    "password": "",
-    "host": "example.org",
-    "hostname": "example.org",
-    "port": "",
-    "pathname": "/test",
-    "search": "?a",
-    "hash": "#bc"
   }
 ]