Closed
Bug 6071
Opened 26 years ago
Closed 25 years ago
Node::nodeValue for comment shouldn't have comment delimeters
Categories
(Core :: DOM: Core & HTML, defect, P3)
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: dbaron, Assigned: waqar)
References
()
Details
Node::nodeValue for Comment nodes shouldn't return the comment delimiters. See
the table in
http://www.w3.org/TR/REC-DOM-Level-1/level-one-core.html#ID-1950641247
but especially the description of "content of a comment" in
http://www.w3.org/TR/REC-DOM-Level-1/level-one-core.html#ID-1728279322
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → DUPLICATE
Comment 1•26 years ago
|
||
Reporter | ||
Updated•26 years ago
|
Status: VERIFIED → REOPENED
Reporter | ||
Comment 3•26 years ago
|
||
Reopening. rickg marked bug 5544 wontfix after I reopened it, giving a reason
that I'm not sure is true anymore. (It doesn't seem to me, through the DOM,
that script or style elements contain any comment nodes anymore. They
shouldn't, either. See bug 7853) I would think it's easier to fix there than
by working around it in the DOM, because there's a lot to work around. However,
you can deal with that ;-). See
http://www.fas.harvard.edu/~dbaron/dom/test/one-core-html/Comment for what needs
doing. (And compare it to
http://www.fas.harvard.edu/~dbaron/dom/test/one-core-xml/Comment .)
Updated•26 years ago
|
Status: REOPENED → ASSIGNED
Target Milestone: M10
Comment 6•26 years ago
|
||
Fix ready to go when tree opens today.
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Comment 7•26 years ago
|
||
Fixed on 8/25/1999.
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 9•25 years ago
|
||
Verified fixed, 1999-10-08-08-M11 Linux apprunner.
Reporter | ||
Updated•25 years ago
|
Status: VERIFIED → REOPENED
Reporter | ||
Comment 10•25 years ago
|
||
This has regressed. Reopening.
I think it would probably be much easier to fix bug 15904 if the delimiters were
not stored in the content model (at least when the delimiters are properly
formed, which is all that really matters for either bug). That is, I think it
would automatically fix said bug.
STEPS TO REPRODUCE:
* load http://www.fas.harvard.edu/~dbaron/dom/test/one-core-html/Comment
* click "test Comment"
EXPECTED RESULTS:
* second, third, and fourth tests green (if you fix this the "right" way, all
the tests should become green)
ACTUAL RESULTS:
* only the first test in the entire page is green
TESTED ON:
* Linux viewer, build from morning of 1999-11-05
Comment 11•25 years ago
|
||
Clearing FIXED resolution due to reopen. Since this was on M10 and that is over,
moving to M11 for consideration.
Comment 12•25 years ago
|
||
Reassigning to Waqar - he has a fix for this.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•25 years ago
|
||
Fix check in, marking it as closed.
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Reporter | ||
Comment 14•25 years ago
|
||
This shows no sign of being fixed in Linux mozilla 1999-11-12-15-M12. Reopening.
Did you *really* check in the fix?
Reporter | ||
Comment 15•25 years ago
|
||
I'm beginning to think the builds on mozilla.org are mislabeled and I'm really
using the build 1999-11-12-18-M11. (I downloaded the file labeled
1999-11-12-15-M12, and the build ID is 1999111218...)
Reporter | ||
Updated•25 years ago
|
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 16•25 years ago
|
||
Re-resolving based on the 1999-11-13-08-M12 build. I think the build I had was
an M11 one.
Reporter | ||
Comment 17•25 years ago
|
||
Marking verified.
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•