Closed Bug 15904 Opened 20 years ago Closed 20 years ago

Problems with text manipulation on comment nodes

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: waqar)

References

()

Details

DESCRIPTION:  There are serious problems using the CharacterData interfaces on
comment nodes.  Basically, any call to insertData(), appendData(), or
deleteData() causes the first character in the comment to disappear.

STEPS TO REPRODUCE:
 * load http://www.fas.harvard.edu/~dbaron/dom/test/one-core-html/Comment
 * run tests

ACTUAL RESULTS:  Lots of red.  Each progressive actual result after the first
section has one fewer character at the beginning.

EXPECTED RESULTS:  Green.

DOES NOT WORK CORRECTLY ON:
 * Linux, apprunner, 1999-10-08-08-M11

ADDITIONAL INFORMATION:
In XML, the problem even shows up for the *initial value* of Comment::data.  See
http://www.fas.harvard.edu/~dbaron/dom/test/one-core-xml/Comment
Assignee: vidur → waqar
Assigning to Waqar for initial investigation.
Status: NEW → ASSIGNED
A fixed checked in. The problem now is Kipps code changes for
nsGenericDOMDataNode::AppendData method. With my changes and the old code it
works fine. Then change I made was to nsCommentNode.cpp.
Target Milestone: M12
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Fix checked in.
Status: RESOLVED → REOPENED
Reopening this one based on Linux mozilla 1999-11-13-08-M12. There's still
something wrong with appendData. That's why two lines in the test page are still
red.
Resolution: FIXED → ---
Reopening this one based on Linux mozilla 1999-11-13-08-M12. There's still
something wrong with appendData. That's why two lines in the test page are still
red.
There is also a problem visible in (the second button of)
http://www.fas.harvard.edu/~dbaron/dom/test/one-core-html/Document .  Could this
have something to do with one-byte versus two-byte characters?
Status: REOPENED → RESOLVED
Closed: 20 years ago20 years ago
Resolution: --- → FIXED
Fix checked in.
Mass update of qa contact
QA Contact: gerardok → janc
Component: DOM Level 1 → DOM HTML
QA contact Update
QA Contact: janc → desale
Updating QA contact to Shivakiran Tummala.
QA Contact: desale → stummala
verified...i do not see any red lines in the testcases
Status: RESOLVED → VERIFIED
Component: DOM: HTML → DOM: Core & HTML
QA Contact: stummala → general
You need to log in before you can comment on or make changes to this bug.