[CSS Grid Layout] ASSERTION: !value || (value->isGridLineNamesValue())

[CSS Grid Layout] ASSERTION: !value || (value->isGridLineNamesValue())

The old Bison parser used to provide a CSSParser::ValueList with the list of
idents that we'll parse as grid line names. The new one instead just provide a
opening bracket followed by 1 or more idents and a closing bracket that must be
parsed by the property parser. The grid-template parsing code was relying on
the existence of one of those CSSParserValue::ValueList to detect invalid
values for that shorthand.

The problem arises whenever we try to detect whether or not the grid
row specification had trailing line names or not. We can no longer look for a
CSSParserValue::List in advance. Instead we take a look at the last correctly
parsed element, if that's a custom indent then we can conclude that the row
specification line indeed has trailing line names.

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

[svillar, 2015-06-09 14:43:25]

svillar@igalia.com changed reviewers:
+ dsinclair@chromium.org, eae@chromium.org, jchaffraix@chromium.org

[svillar, 2015-06-09 14:43:25]

Sending for review

[Julien - ping for review, 2015-06-09 18:28:10]

jchaffraix@chromium.org changed reviewers:
+ timloh@chromium.org

[Julien - ping for review, 2015-06-09 18:28:10]

The CQ bit was checked by jchaffraix@chromium.org

[Julien - ping for review, 2015-06-09 18:28:12]

lgtm

[commit-bot: I haz the power, 2015-06-09 18:28:26]

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

[commit-bot: I haz the power, 2015-06-09 19:35:16]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196794