Stricter parsing for span attribute on col element

Stricter parsing for span attribute on col element

According to the latest spec "The element may have a span content
attribute specified, whose value must be a valid non-negative
integer greater than zero."

With this patch parsing is more strict and above rules are honored.
This was showing as failures here, fixed with this patch:
http://w3c-test.org/html/dom/reflection-forms.html

Behavior matches Firefox.

Spec reference:
https://html.spec.whatwg.org/#dom-colgroup-span

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

[rwlbuis, 2015-01-06 23:48:28]

rob.buis@samsung.com changed reviewers:
+ kouhei@chromium.org, morrita@chromium.org

[rwlbuis, 2015-01-06 23:48:28]

PTAL. Normally I would ask tkent but I think he is OOO. Feel free to assign to
somebody else if that person can do a better review.

[Hajime Morrita, 2015-01-06 23:58:17]

On 2015/01/06 23:48:28, rwlbuis wrote:
> PTAL. Normally I would ask tkent but I think he is OOO. Feel free to assign to
> somebody else if that person can do a better review.
LGTM. This looks too right :-o
Probably it's nice to have a link to the spec in the code,
unless we no longer do it these days.

[kouhei (in TOK), 2015-01-07 05:51:45]

lgtm

[rwlbuis, 2015-01-07 15:10:14]

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

[commit-bot: I haz the power, 2015-01-07 15:11:00]

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

[commit-bot: I haz the power, 2015-01-07 16:26:58]

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