Closed Bug 21007 Opened 25 years ago Closed 25 years ago

Deeply nested tables crashes Mozilla

Categories

(Core :: Layout, defect, P3)

Product:
Core
Component:
Layout
Platform:
x86
Other
Type:
defect

Tracking

()

Status:
VERIFIED WORKSFORME

People

(Reporter: mozilla, Assigned: troy)

References

(
URL
)

Details

Attachments

(1 file)

Deeply nested tables crashing Firefox 20
84.20 KB, text/plain
Details 
OK. This page is ridiculous with the tables, but it was only meant to be a test.
Unfortunately Mozilla fails miserably and so does Communicator. IE5 handles it
fine.

When Mozilla or Netscape is used, the process consumes 100% CPU for more than 10
minutes (I shut it off), IE uses 10% CPU for about 20 seconds.

I warn you that the page is nearly 1MB of nested tables.
jerrybaker@weirdness.com -- perhaps in the future you could provide a bit better
description of such pages (i.e., why make troy or karnaze poke around when the
HTML can be described as easily as below -- then they can decide priorities).

Also, to complete the comparison of browsers --
  IE 4.0 on WinNT with 80MB RAM crashes on a stack overflow after ~15 seconds

Each nested table is composed of 56 repetitions of
  <table width="100%" border="0" cellspacing="0" cellpadding="0"> <tr> <td>
which are then closed by 56 matching '</td> </tr> </table>'
There are 15 such nested tables in the page.
Severity: normal → critical
You're right. I should have noted that it was IE 5 on my machine with 256MB that
did not crash. In fact, it only took 20 seconds because of the file being 1MB.

I know this test case is ridiculous, but it was prompted by a real world problem
with Communicator. A user had created a site with only 13 nested tables which
would lock up Communicator solid. I tested Mozilla and noticed it got slow, but
didn't crash, so I took it to the extreme.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
Cannot make it happen with M12. Appears to be fixed. I even tried 20 tables 35
nested tables each. Mozilla handled it beautifully in 2.9 seconds.
The url is invalid so I can't check for myself. The reporter last comments
mention that the problem has been fixed. Based on the reporter's last commenst,
marking verified fixed in the Dec 22nd build.
Status: RESOLVED → VERIFIED
Sorry. URL works now.
Mass removing self from CC list.
Now I feel sumb because I have to add back. Sorry for the spam.
This file crashes Firefox 20. It is correctly rendered in Chromium.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: