Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511
[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span

Committed: https://crrev.com/bd23011108522aacf45a19bc7f792abf2eece78d
Cr-Commit-Position: refs/heads/master@{#422669}

[rwlbuis, 2016-10-03 21:35:25]

Description was changed from

==========
cols/rows WIP

BUG=
==========

to

==========
cols/rows WIP

BUG=490511
==========

[rwlbuis, 2016-10-03 21:37:46]

Description was changed from

==========
cols/rows WIP

BUG=490511
==========

to

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead.

BUG=490511
==========

[rwlbuis, 2016-10-03 21:40:19]

Description was changed from

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead.

BUG=490511
==========

to

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
==========

[rwlbuis, 2016-10-03 21:41:26]

Description was changed from

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
==========

to

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span
==========

[rwlbuis, 2016-10-03 21:41:40]

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

[rwlbuis, 2016-10-03 21:41:40]

PTAL.

[tkent, 2016-10-03 23:31:26]

lgtm

[rwlbuis, 2016-10-03 23:34:23]

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

[commit-bot: I haz the power, 2016-10-03 23:35:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-04 02:06:57]

Description was changed from

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span
==========

to

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span
==========

[commit-bot: I haz the power, 2016-10-04 02:06:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 02:09:03]

Description was changed from

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span
==========

to

==========
Make cols/rows/span not accept zero when set from idl

Make cols/rows/span not accept zero when set from idl, but do
similar to what happens when setting the attribute, i.e. set
the default values instead [1].

BUG=490511

[1] https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-cols
    https://html.spec.whatwg.org/multipage/forms.html#attr-textarea-rows
    https://html.spec.whatwg.org/multipage/tables.html#dom-colgroup-span

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

[commit-bot: I haz the power, 2016-10-04 02:09:04]

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