Align color keyword handling in CSSParserFastPaths with CSSPropertyParser

Align color keyword handling in CSSParserFastPaths with CSSPropertyParser

Previously there has been some discrepancies between
CSSParserFastPaths::parseColor and CSSPropertyParser::parseColor with
regards which color keywords they accept, and whether they do so in
quirks mode or not. Use the same sequence of isColorKeyword and
isValueAllowedInMode in both places to try to prevent that happening
(again.)

BUG=505410
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=199021

[fs, 2015-07-13 13:59:27]

fs@opera.com changed reviewers:
+ timloh@chromium.org

[fs, 2015-07-13 13:59:28]

https://codereview.chromium.org/1235943002/diff/1/Source/core/css/parser/CSSP...
File Source/core/css/parser/CSSParserFastPaths.cpp (right):

https://codereview.chromium.org/1235943002/diff/1/Source/core/css/parser/CSSP...
Source/core/css/parser/CSSParserFastPaths.cpp:438: if
(!isValueAllowedInMode(valueID, parserMode))
Could also just have picked HTMLStandardMode/HTMLQuirksMode based on the
existing bool (for a smaller change)... Let me know which you prefer.

[Timothy Loh, 2015-07-16 03:20:33]

lgtm

[fs, 2015-07-16 08:28:24]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-07-16 08:28:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1235943002/1

[commit-bot: I haz the power, 2015-07-16 09:44:20]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199021