83605 - Unknown tags in document cause nested table contents to be hidden

Status: RESOLVED WORKSFORME

(Core :: Layout, defect, P3)

Description

[Benjamin Mucci]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9+) Gecko/
BuildID:    2001053108

In the page http://www.perspectives.qc.ca/bugzilla/ (border is set to "1" for
best view) we should see 3 forms that contain 1 txt field + 1 image button.
Everything is correct in NS 4.x and IE but in Mozilla (in every build ID since
long time ago) the content of the table cell (red in the page) is empty (The
form 1 is hidden). I made a copy of the cell content out of the main table (Form
3) and it OK in Moz.

Reproducible: Always
Steps to Reproduce:
1.Just see http://www.perspectives.qc.ca/bugzilla/ in Moz
2.
3.

Actual Results:  No form 1 in the red table cell.

Expected Results:  Form 1 in the red table cell

This problem appear in other pages like:

http://www.perspectives.qc.ca/fr/intervenants/
No text at the right of grey bullets

http://www.clscdesfaubourgs.qc.ca/fr/
Some contents in the top and right parts are hidden

http://www.clsc-mercier-est-anjou.qc.ca/navigation/actualites/agenda.shtml
Text under "Agenda des activités..." and the table at the right.

Comment 1

[Asa Dotzler [:asa]]

updating component.

Comment 2

[Antti Näyhä]

The problem is caused by the non-standard <csaction> tags (unknown to Mozilla)
in the beginning of the document.

Attaching a minimized test case, which has 60 <foo> tags before a simple nested
table. On my Mozilla 0.9 (MacOS 9.0), the text "bar" inside the table is hidden.
Remove any of the <foo> tags and it works.

Interestingly enough, if <csaction> is used instead of <foo>, 59 tags is enough to
show the bug. Looks like a parser problem to me...

Comment 3

[Antti Näyhä]

Attachment: Minimized test case

Comment 4

[Antti Näyhä]

Reassigning to parser guys...

Comment 5

[harishd]

I'm able to see the text "bar" in Netscape 6.1 PR1.

Comment 6

[Benjamin Mucci]

It's better now but not perfect:

http://www.clscdesfaubourgs.qc.ca/fr/territoire/portrait_territoire/

Build 2001062015

Comment 7

[Benjamin Mucci]

same in 2001062708... need some work again I think...

Comment 8

[Benjamin Mucci]

Attachment: "Example 6: Table Stress Test (resource:///res/samples/test6.html) " with unknow tag

Comment 9

[Benjamin Mucci]

Attachment: The actual result under MacOS

Comment 10

[Benjamin Mucci]

Other exemple with lot of MS garbages in code:

http://www.resacorp.com/arithmetic_mean.htm

NS4.77 Mac: Ok.
Moz Win98:  Ok.
Moz Mac:    Failed

Build 2001062823

Comment 11

[Amarendra Hanumanula]

 I think this is a table bug.. Changing this component to HTML Tables and QA
contact to amar
  

Comment 12

[Benjamin Mucci]

http://www.resacorp.com/arithmetic_mean.htm works ok in 2001070904 but "Table
Stress Test + unknow tags" doesn't works properly.


Got other confirmations of the Bug under MacOS 8.6/9 and MacOS X under Classic on:
Powerbook G3 333 MacOS 8.6 (20001070904)
PowerBook Ti 400 MacOS X with Classic (200107070)
8500/400Mhz MacOS 9.1  (20001070803)
G4/400 MacOS 9.1 

Accept bug, set priority and target someone?

Comment 13

[harishd]

> http://www.resacorp.com/arithmetic_mean.htm works ok in 2001070904 but "Table
> Stress Test + unknow tags" doesn't works properly.

Could you please tell me what exactly is the problem?

Comment 14

[Benjamin Mucci]

For demonstration I used the "Example 6: Table Stress Test" from menu "Debug > 
Viewer demos > #6 Deeply nasted tables" added lot of unknown tags (as  Antti 
Näyhä exemple)

"
<foo>
<foo>
<foo>
...
"
file is:
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=40741 (check source)


In NS4.7 Mac/Win, IE5.x Mac/Win and Moz Win this file works just great. But when 
Moz Mac *meet* this kind of "unknown tags": table become mess up. 

Screenshoot of the result in MozMac:

http://bugzilla.mozilla.org/showattachment.cgi?attach_id=40854

In reel life "unknown tags" are used for exemple by Adobe Golive:

********************************************************************************
<csaction name="B359D2C31" class="Set Image URL" type="ONEVENT" val0=
"description_lien" val1="../../../images/descriptions_liens/description_lien-
nouveautes.gif">
<csaction name="B359D2C32" class="Set Image URL" type="ONEVENT" val0="new" val1=
"../../../images/menu_navigation_dynamique/menu_nav-02-02.gif">
<csaction name="B359D2C33" class="Set Image URL" type="ONEVENT" val0=
"description_lien" val1="../../../images/descriptions_liens/description_lien-
blank.gif">
<csaction name="B359D2C34" class="Set Image URL" type="ONEVENT" val0=
"communicate" val1="../../../images/menu_navigation_dynamique/menu_nav_fleches-
02-03.gif">
********************************************************************************               
as you find in the multiple URL I posted.


Will add 2 screenshoots NS4.7 vs Moz:

The first is VERY problematic:
http://www.clscdesfaubourgs.qc.ca/fr/organisation/conseil-administration/
reglements_ca.html

All http://www.clscdesfaubourgs.qc.ca/ except the home page is problematic in 
fact (Under Mac).

Other exemple:
http://www.clsc-mercier-est-anjou.qc.ca/navigation/actualites/agenda.shtml

Comment 15

[Benjamin Mucci]

Attachment: Exemple 1

Comment 16

[Benjamin Mucci]

Attachment: Exemple 2

Comment 17

[Simon Fraser [no longer active]]

This is happening because on Mac we limit the recursion depth in reflow, to
avoid blowing the stack. The real fix would be to make layout non-recursive, or
dramaticallly reduce its stack usage. Similar bugs already exist (bug 55095, bug
43174, bug 42138).

Comment 18

[harishd]

*** This bug has been marked as a duplicate of 61684 ***

Comment 19

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I think we should keep this bug open as a separate bug.  Bug 61684 is about
making us faster while keeping recursive reflow, whereas this bug requires that
we eliminate most of the recursion in reflow.  Reopening.

Comment 20

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

and giving to waterson because he has the other bugs like this (not because I
expect him to wake up one morning and decide to fix it).

Comment 21

[Chris Waterson]

Ouch!

Comment 22

[Benjamin Mucci]

Good news folks: WFM in 2001082404. 

Thank Waterson? ;-)

Comment 23

[Benjamin Mucci]

miraculously disappears -> WFM.