480550 - setting htmlElement.visibility hidden breaks rendering

Status: RESOLVED DUPLICATE of bug 97506

(Firefox :: General, defect)

Description

[igor styrman]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.6) Gecko/2009011913 Firefox/3.0.6 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.6) Gecko/2009011913 Firefox/3.0.6 (.NET CLR 3.5.30729)

With javascript keep setting table/tr style to invisible and back. The TR is rendered with incorrect indentation when it works. If you input text into the corresponding DT-element this produces an empty additional row. 

Below is a test case HTML encoded with base-64:

CjxodG1sPgo8aGVhZD4KPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgoKLy8gPGM6aWZm
YWlsbGFhbiB2YWxpdHR1IG1hYSBxcQpzZXRUaW1lb3V0KCJhZGRBZGRyZXNzMignRkknKSIsIDI1
MCk7CgovKioKICogSm9zIHZhbGl0YWFuIHVsa29tYWEsIG7keXRldOTkbiBsaXPkb3NvaXRla2Vu
dHTkLgogKgogKiBAcGFyYW0gc2VsZWN0ZWRWYWx1ZSBtYWFrb29kaQogKi8KZnVuY3Rpb24gYWRk
QWRkcmVzczIoc3RyU2VsZWN0ZWRWYWx1ZSkKewoJaWYgKHN0clNlbGVjdGVkVmFsdWUgPT0gIkZJ
IikKCXsKCQl2YXIgb2JqRWxlbWVudCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJrYXR1b3Nv
aXRlMiIpOwoJCW9iakVsZW1lbnQuc3R5bGUuZGlzcGxheSA9ICJub25lIjsKCQlvYmpFbGVtZW50
LnN0eWxlLnZpc2liaWxpdHkgPSAiaGlkZGVuIjsJCQoJfQoJZWxzZQoJewoJCXZhciBvYmpFbGVt
ZW50ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImthdHVvc29pdGUyIik7CgkJb2JqRWxlbWVu
dC5zdHlsZS5kaXNwbGF5ID0gImJsb2NrIjsKCQlvYmpFbGVtZW50LnN0eWxlLnZpc2liaWxpdHkg
PSAidmlzaWJsZSI7CQkKCX0KfQo8L3NjcmlwdD4KPC9oZWFkPgo8Ym9keT4KCjxzZWxlY3Qgb25j
aGFuZ2U9ImFkZEFkZHJlc3MyKHRoaXMub3B0aW9uc1t0aGlzLnNlbGVjdGVkSW5kZXhdLnZhbHVl
KTsiPgoJPG9wdGlvbiB2YWx1ZT0icXEiPmZ1YmFyPC9vcHRpb24+Cgk8b3B0aW9uIHZhbHVlPSJu
biI+Zm9vPC9vcHRpb24+Cgk8b3B0aW9uIHZhbHVlPSJGSSIgc2VsZWN0ZWQ9InRydWUiPitTVU9N
SSs8L29wdGlvbj4KPC9zZWxlY3Q+CgoKPGJyLz4KPGJyLz4KCgo8Zm9ybSBuYW1lPSJmb29nIj4K
CTx0YWJsZT4KCQk8dHI+PHRkPgoJCQk8aW5wdXQgbmFtZT0icG9zdGlvc29pdGUiIGlkPSJpZDQi
IHZhbHVlPSIiIHR5cGU9InRleHQiIHNpemU9IjEyOCIvPgoJCTwvdGQ+PC90cj4JCgkJPHRyIGlk
PSJrYXR1b3NvaXRlMiI+PHRkPgoJCQk8aW5wdXQgdHlwZT0idGV4dCIgbmFtZT0icG9zdGlvc29p
dGUyIiB2YWx1ZT0iIiBpZD0iaWQ1IiBzaXplPSIxMjgiLz4KCQk8L3RkPjwvdHI+CgkJPHRyPjx0
ZD4KCQkJPGlucHV0IG5hbWU9InBvc3Rpb3NvaXRlIiB2YWx1ZT0iZHVtZGlkb20iIHR5cGU9InRl
eHQiIHNpemU9IjEyOCIvPgoJCTwvdGQ+PC90cj4KCTwvdGFibGU+CjwvZm9ybT4KCjwvYm9keT4K
Cgo8L2h0bWw+Cg==

Reproducible: Always

Comment 1

[Daniel.S]

While and after creating a bug report, you can add an attachement (a example HTML document for example), see the "Add an attachment  (proposed patch, testcase, etc.)" part just over your comment in the bug report.

Thanks for reporting.

Comment 2

[Mats Palmgren (inactive)]

Bug 480549 is just gibberish so let's use this bug instead.
Igor, please upload a plain HTML testcase using the "Add an attachment" link.

Comment 4

[igor styrman]

Ok sorry for the fuzz, with bug id *49 i was testing if it accepts longer text and suprise suprise it was allready saved..

I'm uploading test case as attachment .html now.

Also tested this with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.6) Gecko/2009020410 Fedora/3.0.6-1.fc10 Firefox/3.0.6 and was able to reproduce.

Note that you have to turn the visibility on and off several times to see it happen. Approximately 5 reps.

Comment 5

[igor styrman]

Atleast on linux the problem is clearly that the white space gap grows between TR's accumulatively.

Comment 6

[igor styrman]

Attachment: test case

Comment 7

[Mats Palmgren (inactive)]

This is a bug in Gecko, but you really should set 'display' to 'table-row'
instead of 'block' for <TR>.  (or the empty string '' should work too,
in case you need a more generic solution)
http://www.w3.org/TR/CSS21/visuren.html#propdef-display