Move isValid/isCurrentColor from Color to StyleColor

Move isValid/isCurrentColor from Color to StyleColor

Change style resolution and RenderStyle to use the new StyleColor class
instead of the platform Color class to represent a color and only
convert to the lower level color class at paint/query time.

This allows the currentColor logic to be contained within RenderStyle
and friends, allows style using currentColor to be cached and shared
and hides the complexity from style resolution. It also moves the
isValid concept from Color into StyleColor, thereby reducing the size
of the lower-level Color class to 32 bits.

The SVG code isn't as clean as I'd like it to be, mainly due to the
fact that SVG has a separate currentColor implementation. I'd like to
merge the two or at least make them work together but that'll have to
follow in a separate change as this one is big enough as is.

BUG=259121
R=eseidel@chromium.org,jamesr@chromium.org
TEST=Covered by existing fast/css tests

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

[eseidel, 2013-07-23 22:12:54]

lgtm

[commit-bot: I haz the power, 2013-07-23 22:13:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/20061003/1

[commit-bot: I haz the power, 2013-07-24 00:52:54]

Change committed as 154797

[tfarina, 2013-07-24 01:16:00]

https://codereview.chromium.org/20061003/diff/1/Source/core/css/StyleColor.cpp
File Source/core/css/StyleColor.cpp (right):

https://codereview.chromium.org/20061003/diff/1/Source/core/css/StyleColor.cp...
Source/core/css/StyleColor.cpp:32: using namespace std;
where are you using std types here? do you need this using directive?

[eae, 2013-07-24 18:47:30]

https://codereview.chromium.org/20061003/diff/1/Source/core/css/StyleColor.cpp
File Source/core/css/StyleColor.cpp (right):

https://codereview.chromium.org/20061003/diff/1/Source/core/css/StyleColor.cp...
Source/core/css/StyleColor.cpp:32: using namespace std;
On 2013/07/24 01:16:00, tfarina wrote:
> where are you using std types here? do you need this using directive?

Good catch, left over from an earlier iteration. I'll remove it.