Hashless hex color quirk for color/background/border only.

Hashless hex color quirk for color/background/border only.

According to the quirks mode spec[1], the hashless hex color quirk only
applies to a few properties including color, background-color (not from
short-hand), and various border-color properties.

This aligns the Blink implementation with Gecko and the spec.

The duplicated code and early break is there to avoid regressions in blink
performance tests (there were cases in the CSS and SVG directories that
detected regressions locally).

[1] http://quirks.spec.whatwg.org/#the-hashless-hex-color-quirk

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

[Timothy Loh, 2014-07-15 04:13:42]

On 2014/07/07 17:48:35, rune (vacation) wrote:

I think this would be simpler if we added a bool flag defaulting to false to
parseColor and parseColorFromValue instead of having a QuirkyColorScope.

[rune, 2014-08-04 23:14:35]

On 2014/07/15 04:13:42, Timothy Loh wrote:
> On 2014/07/07 17:48:35, rune (vacation) wrote:
> 
> I think this would be simpler if we added a bool flag defaulting to false to
> parseColor and parseColorFromValue instead of having a QuirkyColorScope.

Done.

[Timothy Loh, 2014-08-05 07:35:42]

On 2014/08/04 23:14:35, rune wrote:
> On 2014/07/15 04:13:42, Timothy Loh wrote:
> > On 2014/07/07 17:48:35, rune (vacation) wrote:
> > 
> > I think this would be simpler if we added a bool flag defaulting to false to
> > parseColor and parseColorFromValue instead of having a QuirkyColorScope.
> 
> Done.

lgtm

[rune, 2014-08-05 07:42:29]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-08-05 07:46:03]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rune@opera.com/373603004/20001

[rune, 2014-08-05 11:01:33]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-08-05 11:02:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rune@opera.com/373603004/40001

[commit-bot: I haz the power, 2014-08-05 16:46:49]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/20922)

[commit-bot: I haz the power, 2014-08-05 21:04:37]

Change committed as 179524