Parse webkit-text-stroke shorthand in CSSPropertyParser with CSSParserTokens

Parse webkit-text-stroke shorthand in CSSPropertyParser with CSSParserTokens

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

BUG=499780
Committed: https://crrev.com/18669d2fac93b3f8880c80277aa38d400f4f4574
Cr-Commit-Position: refs/heads/master@{#359423}

[rwlbuis, 2015-11-03 19:35:17]

Patchset #2 (id:20001) has been deleted

[rwlbuis, 2015-11-09 15:30:20]

Description was changed from

==========
text-stroke WIP


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.

BUG=499780

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

to

==========
Parse webkit-text-stroke shorthand in CSSPropertyParser with CSSParserTokens

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

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

[rwlbuis, 2015-11-09 16:45:41]

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

[rwlbuis, 2015-11-09 16:45:42]

PTAL. The different test expectation is because consumeShorthandGreedily adds
properties according the the shorthand array, whereas (legacy) parseShorthand
adds whenever it parses a suitable value. I don't really think one way is better
than the other.

[Timothy Loh, 2015-11-12 20:36:03]

On 2015/11/09 16:45:42, rwlbuis wrote:
> PTAL. The different test expectation is because consumeShorthandGreedily adds
> properties according the the shorthand array, whereas (legacy) parseShorthand
> adds whenever it parses a suitable value. I don't really think one way is
better
> than the other.

lgtm

[rwlbuis, 2015-11-12 21:35:45]

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

[commit-bot: I haz the power, 2015-11-12 21:40:07]

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

[commit-bot: I haz the power, 2015-11-12 23:12:22]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2015-11-12 23:13:12]

Patchset 4 (id:??) landed as
https://crrev.com/18669d2fac93b3f8880c80277aa38d400f4f4574
Cr-Commit-Position: refs/heads/master@{#359423}