Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.

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

[rwlbuis, 2015-12-15 21:55:16]

Description was changed from

==========
list-style WIP


Move content property into CSSPropertyParser

Move content property handling from LegacyCSSPropertyParser
into CSSPropertyParser.

BUG=499780

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

to

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.

This CL adds the minimal fill image functionality to not regress
the layout tests.

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

[rwlbuis, 2015-12-15 21:56:49]

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

[rwlbuis, 2015-12-15 21:56:49]

PTAL. Hopefully this helps regarding the issue from the spreadsheet:
~~move fastpath keyword callsite into parseSingleValue

[rwlbuis, 2015-12-15 21:57:33]

Description was changed from

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.

This CL adds the minimal fill image functionality to not regress
the layout tests.

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

to

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.


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

[Timothy Loh, 2015-12-16 00:21:49]

lgtm

On 2015/12/15 21:56:49, rwlbuis wrote:
> PTAL. Hopefully this helps regarding the issue from the spreadsheet:
> ~~move fastpath keyword callsite into parseSingleValue

It should :)

[rwlbuis, 2015-12-16 00:39:24]

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

[commit-bot: I haz the power, 2015-12-16 00:40:35]

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

[commit-bot: I haz the power, 2015-12-16 02:48:32]

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

[commit-bot: I haz the power, 2015-12-16 02:48:33]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[Timothy Loh, 2015-12-16 03:18:38]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2015-12-16 03:21:20]

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

[commit-bot: I haz the power, 2015-12-16 04:19:12]

Description was changed from

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.


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

to

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.


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

[commit-bot: I haz the power, 2015-12-16 04:19:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-16 04:20:49]

Description was changed from

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.


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

to

==========
Parse list-style shorthand in CSSPropertyParser with CSSParserTokens

Move list-style property handling from LegacyCSSPropertyParser
into CSSPropertyParser. Also move its related longhands.

BUG=499780

Committed: https://crrev.com/b35c553620dce166c62f02859ec158ebef19c63f
Cr-Commit-Position: refs/heads/master@{#365438}
==========

[commit-bot: I haz the power, 2015-12-16 04:20:49]

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