Closed
Bug 11246
Opened 25 years ago
Closed 25 years ago
Crash on <tree> with border-collapse style
Categories
(Core :: XUL, defect, P3)
Tracking
()
VERIFIED
FIXED
M10
People
(Reporter: enndeakin, Assigned: buster)
Details
Attachments
(1 file)
(deleted),
text/plain
|
Details |
The XUL code below causes a crash on both the Mac and Win95 M8 builds and
possibly others.
<?xml version="1.0"?>
<?xml-stylesheet href="xul.css" type="text/css"?>
<window id="zzz" xmlns="http://www.mozilla.org/keymaster/gatekeeper/
there.is.only.xul">
<tree style="border-collapse: collapse;">
<treehead>
<treerow>
<treecell>Col 1</treecell>
</treerow>
</treehead>
<treechildren>
<treerow>
<treecell>1</treecell>
</treerow>
</treechildren>
</tree>
</window>
Removing the style attribute from the tree tag makes it work fine. Note that I
don't know whether border-collapse is allowed on trees.
I tried an HTML table equivalent of the above code, adjusted it in a number of
different ways, but could not get it to crash. So I'm guessing that this is
isolated to XUL trees.
I was testing the code above from a file I created in the res/samples directory.
Updated•25 years ago
|
Assignee: trudelle → hyatt
Comment 1•25 years ago
|
||
I don't know if that's valid either, although it shouldn't crash. reassigning
to hyatt for triage.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 2•25 years ago
|
||
It's valid. It shouldn't crash.
Updated•25 years ago
|
Assignee: hyatt → buster
Status: ASSIGNED → NEW
Comment 3•25 years ago
|
||
Dup of a bug buster has. Giving it to him.
cc hyatt. David, I think I have a fix for this. What is the syntax for loading
a XUL test case like this into apprunner so I can test it? Thanks!
Comment 5•25 years ago
|
||
Don't worry about it. I saw this crash on tables as well as trees, so if you've
plugged the problem with tables, I'll bet trees are safe. The guy who reported
the bug can handle verifying it with his test case (or better yet he can attach
it to the bug so you can try it out).
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 7•25 years ago
|
||
Comment 8•25 years ago
|
||
Verified fixed on:
1999082412 MacOS85
Also tested (verified fixed) on:
1999082412 WinNT
1999082408 Linux6
Status: RESOLVED → VERIFIED
BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL. XUL
component will be deleted.
Component: XUL → XP Toolkit/Widgets: XUL
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: ckritzer → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•