Closed
Bug 11321
Opened 27 years ago
Closed 26 years ago
appending a TD to a TR causes incorrect frame creation
Categories
(Core :: DOM: Core & HTML, defect, P1)
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: rcv64032, Assigned: karnaze)
Details
(Whiteboard: [TESTCASE] crashes just as described)
Attachments
(2 files)
<HTML>
<BODY>
<SCRIPT>
tbl = document.body.appendChild(document.createElement("TABLE"));
alert("Appending TR to TABLE");
tr = tbl.appendChild(document.createElement("TR"));
alert("Creating TD");
tmp_td = document.createElement("TD")
alert("Appending TD to TR");
td1 = tr.appendChild(tmp_td); // <-- Causes Crash
alert("Adding text");
td1_text = td1.appendChild(document.createTextNode("testing"));
</SCRIPT>
</BODY>
</HTML>
| Reporter | ||
Comment 1•27 years ago
|
||
Comment 2•27 years ago
|
||
Updated•27 years ago
|
Whiteboard: [TESTCASE] crashes just as described
Comment 3•27 years ago
|
||
On the 19990805 M9 Win98 build, nothing happens when I click the button on the
original testcase. It seems like maybe alert() isn't working. Anyway, I created
a new non-alert()-calling version of the testcase. Bang! This one crashes.
Updated•26 years ago
|
Severity: normal → critical
Priority: P3 → P1
Target Milestone: M11
Comment 4•26 years ago
|
||
Crashes are all M11/P1/critical.
Updated•26 years ago
|
Assignee: vidur → karnaze
Summary: appending a TD to a TR causes a crash → appending a TD to a TR causes incorrect frame creation
Comment 5•26 years ago
|
||
The good news is that we don't crash anymore. The bad news is that there are
frame construction problems with tables in specific cases. The test cases
included above create a reasonable content subtree, but it seems like the
inner text frame is created incorrectly. The test does work, however, if the
entire table subtree is created before appending it to the document i.e. the
script is changed to:
function run(){
tbl = document.createElement("TABLE");
alert("Appending TR to TABLE");
tr = tbl.appendChild(document.createElement("TR"));
alert("Creating TD");
tmp_td = document.createElement("TD")
alert("Appending TD to TR");
td1 = tr.appendChild(tmp_td); // <-- Causes Crash
alert("Adding text");
td1_text = td1.appendChild(document.createTextNode("testing"));
document.body.appendChild(tbl);
}
| Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
| Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 6•26 years ago
|
||
Fixed with latest checkin.
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 7•26 years ago
|
||
verified
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•