[css-grid] Consider invalid using just "span" as grid-line value

[css-grid] Consider invalid using just "span" as grid-line value

The spec is pretty clear about this:
https://drafts.csswg.org/css-grid/#typedef-grid-row-start-grid-line

The possible options when using "span" to reference a grid line are:
  [ span && [ <integer> || <custom-ident> ] ]

Thus, "span" keyword alone produces an invalid declaration.

Add test case in grid-item-column-row-get-set.html.
Updated other tests that were using "span" alone, with 2 approaches:
* In some of them changing "span" by "span 1" making it valid.
* In other changing the expected result to "auto", as it's invalid now.

BUG=563400
TEST=fast/css-grid-layout/grid-item-column-row-get-set.html

Committed: https://crrev.com/550b6b9f75065106d09aed429937a9c7b4b31b87
Cr-Commit-Position: refs/heads/master@{#393202}

[Manuel Rego, 2016-05-11 21:21:05]

rego@igalia.com changed reviewers:
+ jfernandez@igalia.com, rob.buis@samsung.com, svillar@igalia.com,
timloh@chromium.org

[rwlbuis, 2016-05-11 21:42:56]

PTAL (assuming it shows up all green).

[rwlbuis, 2016-05-11 22:25:20]

On 2016/05/11 21:42:56, rwlbuis wrote:
> PTAL (assuming it shows up all green).

Arg, I meant LGTM :)

[Manuel Rego, 2016-05-12 06:42:04]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-05-12 06:42:11]

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

[commit-bot: I haz the power, 2016-05-12 07:52:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-12 07:54:00]

Description was changed from

==========
[css-grid] Consider invalid using just "span" as grid-line value

The spec is pretty clear about this:
https://drafts.csswg.org/css-grid/#typedef-grid-row-start-grid-line

The possible options when using "span" to reference a grid line are:
  [ span && [ <integer> || <custom-ident> ] ]

Thus, "span" keyword alone produces an invalid declaration.

Add test case in grid-item-column-row-get-set.html.
Updated other tests that were using "span" alone, with 2 approaches:
* In some of them changing "span" by "span 1" making it valid.
* In other changing the expected result to "auto", as it's invalid now.

BUG=563400
TEST=fast/css-grid-layout/grid-item-column-row-get-set.html
==========

to

==========
[css-grid] Consider invalid using just "span" as grid-line value

The spec is pretty clear about this:
https://drafts.csswg.org/css-grid/#typedef-grid-row-start-grid-line

The possible options when using "span" to reference a grid line are:
  [ span && [ <integer> || <custom-ident> ] ]

Thus, "span" keyword alone produces an invalid declaration.

Add test case in grid-item-column-row-get-set.html.
Updated other tests that were using "span" alone, with 2 approaches:
* In some of them changing "span" by "span 1" making it valid.
* In other changing the expected result to "auto", as it's invalid now.

BUG=563400
TEST=fast/css-grid-layout/grid-item-column-row-get-set.html

Committed: https://crrev.com/550b6b9f75065106d09aed429937a9c7b4b31b87
Cr-Commit-Position: refs/heads/master@{#393202}
==========

[commit-bot: I haz the power, 2016-05-12 07:54:01]

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