[webkit-reviews] review granted: [Bug 15360] Invalid color #{predefined colorName} is accepted by the CSS parser. : [Attachment 88275] proposed patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Andras Becsi <abecsi at webkit.org>'s
request for review:
Bug 15360: Invalid color #{predefined colorName} is accepted by the CSS parser.
https://bugs.webkit.org/show_bug.cgi?id=15360

Attachment 88275: proposed patch
https://bugs.webkit.org/attachment.cgi?id=88275&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=88275&action=review

It’s kind of wasteful that the CSS parser already knows it’s hex in the
hexcolor case, but still does so much work to process other color formats. Not
new, of course, but could be so much more efficient given the lexer has already
told us it's a hex number.

> Source/WebCore/css/tokenizer.flex:49
> +"#"{hexnum}		   {yyTok = HEX; return yyTok;}

I think {h}+ would read just as well as {hexnum} if not better, so I don’t
think we really need hexnum.