Chromium Code Reviews| Index: Source/core/loader/LinkHeader.cpp |
| diff --git a/Source/core/loader/LinkHeader.cpp b/Source/core/loader/LinkHeader.cpp |
| index e9adad68e923b3ac3f845d7737d073365f95aa23..163bf58bb04a1479990f95a5e07ea1ebd31e04a3 100644 |
| --- a/Source/core/loader/LinkHeader.cpp |
| +++ b/Source/core/loader/LinkHeader.cpp |
| @@ -25,7 +25,7 @@ static bool isValidURLChar(CharType chr) |
| template <typename CharType> |
| static bool isValidParameterNameChar(CharType chr) |
| { |
| - return !isWhitespace(chr) && chr != '='; |
| + return !isWhitespace(chr) && chr != '=' && chr != ';' && chr != ','; |
|
Nate Chapin
2015/08/27 21:39:16
Is there a spec for this?
|
| } |
| template <typename CharType> |
| @@ -40,6 +40,7 @@ static bool isValidParameterValueChar(CharType chr) |
| return !isWhitespace(chr) && !isValidParameterValueEnd(chr); |
| } |
| +// Verify that the parameter is a link-extension which according to spec doesn't have to have a value. |
| static bool isExtensionParameter(LinkHeader::LinkParameterName name) |
| { |
| return name > LinkHeader::LinkParameterAnchor; |