Reject invert for outline-color at parse time.

Reject invert for outline-color at parse time.

We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399
Committed: https://crrev.com/1e71d3d9b4687a515e0aca0089a46cc8df84ce84
Cr-Commit-Position: refs/heads/master@{#400902}

[rune, 2016-06-20 08:45:18]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-20 08:45:24]

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

[rune, 2016-06-20 08:48:13]

ptal

[commit-bot: I haz the power, 2016-06-20 08:54:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-20 08:54:23]

Dry run: Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[rune, 2016-06-20 09:02:42]

I misread the spec. We should reject the whole declaration at parse time if we
don't support invert. That's what Gecko does as well.

[rune, 2016-06-20 09:57:04]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-20 09:57:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2081633002/20001

[commit-bot: I haz the power, 2016-06-20 11:13:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-20 11:13:46]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rune, 2016-06-20 11:18:07]

Description was changed from

==========
Render invert as currentColor for outline.

According to the spec, we should render invert as currentColor if we
don't render outlines as inverted. There was already code for that in
TextLinkColors::colorFromCSSValue, but we passed in Color() for
currentColor. Pass in the currentColor and use that codepath for the
currentColor ident as well.

R=timloh@chromium.org
BUG=620399
==========

to

==========
Reject invert for outline-color at parse time.
    
We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399
==========

[rune, 2016-06-20 11:18:37]

Ready for review again. The chromeos failure looks unrelated.

[Timothy Loh, 2016-06-21 00:13:57]

On 2016/06/20 11:18:37, rune wrote:
> Ready for review again. The chromeos failure looks unrelated.

lgtm

[rune, 2016-06-21 05:18:30]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-06-21 05:18:47]

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

[commit-bot: I haz the power, 2016-06-21 05:49:47]

Description was changed from

==========
Reject invert for outline-color at parse time.
    
We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399
==========

to

==========
Reject invert for outline-color at parse time.
    
We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399
==========

[commit-bot: I haz the power, 2016-06-21 05:49:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-21 05:51:54]

Description was changed from

==========
Reject invert for outline-color at parse time.
    
We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399
==========

to

==========
Reject invert for outline-color at parse time.

We don't support invert as outline-color, so we should drop it at parse
time as per spec[1]. Added test to check that the initial value of
outline-color is the computed value of currentColor.

Gecko also drops declarations with invert for outline-color.

[1] https://drafts.csswg.org/css2/ui.html#value-def-invert

R=timloh@chromium.org
BUG=620399

Committed: https://crrev.com/1e71d3d9b4687a515e0aca0089a46cc8df84ce84
Cr-Commit-Position: refs/heads/master@{#400902}
==========

[commit-bot: I haz the power, 2016-06-21 05:51:55]

Patchset 2 (id:??) landed as
https://crrev.com/1e71d3d9b4687a515e0aca0089a46cc8df84ce84
Cr-Commit-Position: refs/heads/master@{#400902}