Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Separate values in Headers with ", ", not ","
Tests expect header values to be separated by that so this fixes a few
of them.
  • Loading branch information
Eijebong committed May 5, 2020
1 parent 2471e9d commit e18a3ad
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 15 deletions.
2 changes: 1 addition & 1 deletion components/script/dom/headers.rs
Expand Up @@ -96,7 +96,7 @@ impl HeadersMethods for Headers {
.get(HeaderName::from_str(&valid_name).unwrap())
{
combined_value = v.as_bytes().to_vec();
combined_value.push(b',');
combined_value.extend(b", ");
}
combined_value.extend(valid_value.iter().cloned());
match HeaderValue::from_bytes(&combined_value) {
Expand Down
14 changes: 0 additions & 14 deletions tests/wpt/metadata/fetch/api/headers/headers-combine.html.ini

This file was deleted.

0 comments on commit e18a3ad

Please sign in to comment.