Don't parse colors for <canvas> in quirks mode

Source/core/html/canvas/CanvasStyle.cpp

Index: Source/core/html/canvas/CanvasStyle.cpp
diff --git a/Source/core/html/canvas/CanvasStyle.cpp b/Source/core/html/canvas/CanvasStyle.cpp
index 28877cd99ef4debd23e9d4d7ee2e606db7cac509..d02914b3b0847a667bf3f88f86e55b35170fd626 100644
--- a/Source/core/html/canvas/CanvasStyle.cpp
+++ b/Source/core/html/canvas/CanvasStyle.cpp
@@ -46,7 +46,7 @@ static ColorParseResult parseColor(RGBA32& parsedColor, const String& colorStrin
 {
     if (equalIgnoringCase(colorString, "currentcolor"))
         return ParsedCurrentColor;
-    if (BisonCSSParser::parseColor(parsedColor, colorString))
+    if (BisonCSSParser::parseColor(parsedColor, colorString, true /* strict */))

[Justin Novosad, 2014/04/16 16:26:49]

The Blinky way of documenting the meaning of 'true' would be to use an enum
instead of a bool. But since this is a pre-existing function, the alternative
would be to declare a constant

const bool useStrictParsing = true;
if (BisonCSSParser::parseColor(parsedColor, colorString, useStrictParsing))

[fs, 2014/04/16 16:34:10]

Ah, yes... I keep forgetting that rule (which think is nice.) Fixed.

         return ParsedRGBA;
     if (BisonCSSParser::parseSystemColor(parsedColor, colorString))
         return ParsedSystemColor;