tbody.deleteRow(-1) should never throw an exception

tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
Committed: https://crrev.com/0c4a630b476aa829cacd0b8fb7d716d5d993692e
Cr-Commit-Position: refs/heads/master@{#379771}

[ramya.v, 2016-03-08 05:36:44]

ramya.v@samsung.com changed reviewers:
+ tkent@chromium.org

[ramya.v, 2016-03-08 05:36:45]

PTAL!

Thanks

[tkent, 2016-03-08 05:48:12]

> parent. Should not throw exception in this case if row is empty.

Oh, I see.  So the problem is not deleteRow(-1) support, but throwing an
exception when there is no rows.
Please update the first line of the CL description.  It should be something like
"tbody.deleteRow(-1) should never throw an exception."

lgtm

[ramya.v, 2016-03-08 05:50:45]

Description was changed from

==========
tbody.deleteRow(-1) should remove the last row

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
==========

to

==========
tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
==========

[ramya.v, 2016-03-08 05:51:00]

The CQ bit was checked by ramya.v@samsung.com

[commit-bot: I haz the power, 2016-03-08 05:51:08]

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

[commit-bot: I haz the power, 2016-03-08 07:11:39]

Description was changed from

==========
tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
==========

to

==========
tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
==========

[commit-bot: I haz the power, 2016-03-08 07:11:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-08 07:12:50]

Description was changed from

==========
tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817
==========

to

==========
tbody.deleteRow(-1) should never throw an exception

As per spec
https://html.spec.whatwg.org/multipage/tables.html#dom-tbody-deleterow
If index is -1, remove the last element in the rows collection from its
parent. Should not throw exception in this case if row is empty.

BUG=592817

Committed: https://crrev.com/0c4a630b476aa829cacd0b8fb7d716d5d993692e
Cr-Commit-Position: refs/heads/master@{#379771}
==========

[commit-bot: I haz the power, 2016-03-08 07:12:52]

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