[Webkit-unassigned] [Bug 15360] color:#{predefined colorName} is okay in Safari

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=15360


wilsong at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22774|                            |review?
               Flag|                            |




------- Comment #3 from wilsong at gmail.com  2008-08-13 09:01 PDT -------
Created an attachment (id=22774)
 --> (https://bugs.webkit.org/attachment.cgi?id=22774&action=view)
Potential patch for bug 15360

In the bison template CSSGrammar.y, "hexcolor" was defined as both "HEX
maybe_space" OR "IDENT maybe_space".  This caused identifiers not fitting the
appropriate hex format, but preceeded by a '#', to be interpreted as a valid
color (CSSPrimitiveValue::CSS_PARSER_HEXCOLOR), when it was really just an
ignorable token.

To correct this, "IDENT maybe_space" was removed from "hexcolor" and added
under "term" as "'#' IDENT maybe_space", which is then processed as a
CSSPrimitiveValue::CSS_STRING instead of
CSSPrimitiveValue::CSS_PARSER_HEXCOLOR.  Thus, the value will still be
processable in CSSParser in the future, but as a CSS_STRING and not a
CSS_PARSER_HEXCOLOR or CSS_PARSER_IDENT.

"IDENT maybe_space" is already being matched earlier in "term", and "'#'
maybe_space" is matched after, so both edge cases are already being matched.

The expected value for the layout test is added under platform/win right now,
but I'd also like to be able to add the expected value for platform/mac next.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.