HTMLTableRowElement.insertCell()'s argument default value should be -1

HTMLTableRowElement.insertCell()'s argument default value should be -1

HTMLTableRowElement.insertCell()'s argument default value should be -1, not
0, as per the specification:
http://www.whatwg.org/specs/web-apps/current-work/multipage/tabular-data.html#the-tr-element

The new behavior matches the one of Firefox 29 and IE11.

R=arv@chromium.org, eseidel@chromium.org
BUG=369803
TEST=fast/dom/HTMLTableRowElement/insertCell-default-argument.html

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

[Inactive, 2014-05-06 17:40:04]

Corresponding WebKit bug:
https://bugs.webkit.org/show_bug.cgi?id=132615

[arv (Not doing code reviews), 2014-05-06 18:19:50]

https://codereview.chromium.org/263323004/diff/1/Source/core/html/HTMLTableRo...
File Source/core/html/HTMLTableRowElement.cpp (right):

https://codereview.chromium.org/263323004/diff/1/Source/core/html/HTMLTableRo...
Source/core/html/HTMLTableRowElement.cpp:143: n = firstChild();
This should be firstElementChild as well.

[Inactive, 2014-05-06 18:21:38]

https://codereview.chromium.org/263323004/diff/1/Source/core/html/HTMLTableRo...
File Source/core/html/HTMLTableRowElement.cpp (right):

https://codereview.chromium.org/263323004/diff/1/Source/core/html/HTMLTableRo...
Source/core/html/HTMLTableRowElement.cpp:143: n = firstChild();
On 2014/05/06 18:19:50, arv wrote:
> This should be firstElementChild as well.

Do you mind if I fix those in a separate patch? I think they are separate
issues.

[arv (Not doing code reviews), 2014-05-06 18:33:14]

LGTM

I second patch seems like the right thing to do.

[Inactive, 2014-05-06 19:47:45]

On 2014/05/06 18:33:14, arv wrote:
> LGTM
> 
> I second patch seems like the right thing to do.

eseidel, can you please take a look as well (needs API OWNER review)?

[eseidel, 2014-05-06 19:56:19]

lgtm

[Inactive, 2014-05-06 19:57:05]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-05-06 19:57:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/263323004/1

[commit-bot: I haz the power, 2014-05-06 20:11:19]

Change committed as 173422