368932 - Repeatedly using appendChild with rows in a empty table breaks layout

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Marc Diethelm]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/20070131 Minefield/3.0a2pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a2pre) Gecko/20070131 Minefield/3.0a2pre

Each time you append a new row into a previously emptied table the row is displayed lower on the page.

Reproducible: Always

Steps to Reproduce:
use testcase

1. remove all children in table element.
2. append a row with cells into the table.
3. repeat.
Actual Results:  
Each time the row is added to the table it's displayed lower on the page.
Yet nothing unexpected is visible in DOM Inspector.

Expected Results:  
It should appear in the same spot every time.

This doesn't occur with DIVs, only with tables.

Discovered this in FF 2.0.0.1. Also found it happens in the latest nightly. Hope this isn't some stupid mistake of mine, but really a gecko/dom/layout/whatever bug. :))

Comment 1

[Marc Diethelm]

Attachment: Testcase demonstrating the problem

Comment 2

[Jesse Ruderman]

Confirmed with Mac trunk.

I think I've run into this bug, but I never took the time to make a reduced testcase.  Thanks :)

Comment 3

[Marc Diethelm]

I'm still seeing this in

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1a1pre) Gecko/2008070102 Minefield/3.1a1pre

Someone with editbug care to confirm this? Thanks.

Comment 4

[Martijn Wargers (dead)]

It's already confirmed. My guess would be this depends on a fix for bug 148810.

Comment 5

[Marc Diethelm]

> It's already confirmed.

Sure. What I meant is: change bug status to CONFIRMED.

Comment 6

[Martijn Wargers (dead)]

There is no bug status CONFIRMED, that is what NEW means and this bug is currently in the NEW state.

Comment 7

[Marc Diethelm]

/me crawls into hole... sorry for the spam.

Comment 8

[Stellan Klebom]

(In reply to comment #1)
> Created an attachment (id=253571) [details]
> Testcase demonstrating the problem

Running the testcase on IE7 results in "Unknown runtime error" on line 15

   document.getElementById("targettablebody").innerHTML = "";

Obviously IE don't like a <table> without child nodes...

Is it really allowed to add a <tr> element as a child to a <table> element?
An implicit <tbody> is always generated from HTML.

Comment 9

[Boris Zbarsky [:bzbarsky]]

Fixed by checkin for bug 148810.  Added tests.