Chromium Code Reviews| Index: Source/core/css/CSSParser.cpp |
| diff --git a/Source/core/css/CSSParser.cpp b/Source/core/css/CSSParser.cpp |
| index 1a3dedcc7d8134eb11b45028a266a09274da0037..be417b95aafbdbb2bacb7dc1dba219a055bc7135 100644 |
| --- a/Source/core/css/CSSParser.cpp |
| +++ b/Source/core/css/CSSParser.cpp |
| @@ -9341,6 +9341,16 @@ static inline bool isEqualToCSSIdentifier(CharacterType* cssString, const char* |
| } |
| template <typename CharacterType> |
| +static inline bool isEqualToCSSCaseSensitiveIdentifier(CharacterType* string, const char* constantString) |
| +{ |
|
alancutter (OOO until 2018)
2013/05/29 13:19:44
I would add an ASSERT(*constantString) here lest t
|
| + do { |
| + if (*string++ != *constantString++) |
| + return false; |
| + } while (*constantString); |
| + return true; |
| +} |
| + |
| +template <typename CharacterType> |
| static CharacterType* checkAndSkipEscape(CharacterType* currentCharacter) |
| { |
| // Returns with 0, if escape check is failed. Otherwise |
| @@ -10258,7 +10268,7 @@ inline bool CSSParser::detectCSSVariablesToken(int length) |
| if (length < sizeof("-webkit-var-*") - 1) |
| return false; |
| CharacterType* name = tokenStart<CharacterType>(); |
| - return name[11] == '-' && isIdentifierStartAfterDash(name + 12) && isEqualToCSSIdentifier(name + 1, "webkit-var"); |
| + return name[11] == '-' && isIdentifierStartAfterDash(name + 12) && isEqualToCSSCaseSensitiveIdentifier(name + 1, "webkit-var"); |
| } |