Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Move text-decoration shorthand property handling from
LegacyCSSPropertyParser into CSSPropertyParser. Also
move its related longhands.

This patch adds a generic consumeShorthandGreedily that can be
used to convert other shorthands.

Finally the patch refuses duplicate keyword entries for text-decoration-line
(like "blink blink") and adds tests for this to text-decoration-line.html.

BUG=499780, 551638
Committed: https://crrev.com/aba3cd0fad04fb412eeba2f93f1760242349bcee
Cr-Commit-Position: refs/heads/master@{#358232}

[rwlbuis, 2015-11-05 21:31:38]

Description was changed from

==========
Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Move text-decoration shorthand property handling from
LegacyCSSPropertyParser into CSSPropertyParser. Also
move its related longhands.

This patch adds a generic consumeShorthandGreedily that can be
used to convert other shorthands.

Finally the patch refuses duplicate keyword entries for text-decoration-line
(like "blink blink") and adds tests for this to text-decoration-line.html.

BUG=499780

Committed: https://crrev.com/df53d14ebf099ad282ccbdd066e892bb6e5cfdaf
Cr-Commit-Position: refs/heads/master@{#357842}

patch from issue 1412803007 at patchset 220001
(http://crrev.com/1412803007#ps220001)
==========

to

==========
Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Move text-decoration shorthand property handling from
LegacyCSSPropertyParser into CSSPropertyParser. Also
move its related longhands.

This patch adds a generic consumeShorthandGreedily that can be
used to convert other shorthands.

Finally the patch refuses duplicate keyword entries for text-decoration-line
(like "blink blink") and adds tests for this to text-decoration-line.html.

BUG=499780
==========

[rwlbuis, 2015-11-05 21:40:16]

Description was changed from

==========
Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Move text-decoration shorthand property handling from
LegacyCSSPropertyParser into CSSPropertyParser. Also
move its related longhands.

This patch adds a generic consumeShorthandGreedily that can be
used to convert other shorthands.

Finally the patch refuses duplicate keyword entries for text-decoration-line
(like "blink blink") and adds tests for this to text-decoration-line.html.

BUG=499780
==========

to

==========
Parse text-decoration shorthand in CSSPropertyParser with CSSParserTokens

Move text-decoration shorthand property handling from
LegacyCSSPropertyParser into CSSPropertyParser. Also
move its related longhands.

This patch adds a generic consumeShorthandGreedily that can be
used to convert other shorthands.

Finally the patch refuses duplicate keyword entries for text-decoration-line
(like "blink blink") and adds tests for this to text-decoration-line.html.

BUG=499780, 551638
==========

[rwlbuis, 2015-11-05 21:57:27]

rob.buis@samsung.com changed reviewers:
+ timloh@chromium.org

[rwlbuis, 2015-11-05 21:57:27]

PTAL. This patch brings back the reverted patch and has the fix for Bug 551638.

[Timothy Loh, 2015-11-05 23:59:22]

On 2015/11/05 21:57:27, rwlbuis wrote:
> PTAL. This patch brings back the reverted patch and has the fix for Bug
551638.

lgtm

[rwlbuis, 2015-11-06 00:58:10]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2015-11-06 00:58:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1405273005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1405273005/1

[commit-bot: I haz the power, 2015-11-06 02:05:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-06 02:06:44]

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