Closed Bug 43863 Opened 25 years ago Closed 24 years ago

PERF: nsIDOMNode::InsertBefore() slow for document fragment insertion

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Status:
VERIFIED DUPLICATE of bug 35294
Milestone:
mozilla0.9.1

People

(Reporter: mozeditor, Assigned: jst)

References

Details

(Keywords: dom1, perf)

Quoting David Baron from bug 35294: "looking at the current implementation of insertBefore, it may not be optimized for inserting documentFragments, something that probably could be done. In particular, it calls insertChildAt for every child inserted. That implementation *is* a peformance bug that could be improved (although it might be a little messy)."
adding dbaron to cc; marking this bug as a blocker for 35294
Blocks: 35294
Keywords: perf
No longer blocks: 35294
Changing to DOM L1 component, cc:ing vidur.
Assignee: clayton → jst
Component: Layout → DOM Level 1
QA Contact: petersen → gerardok
jst, vidur: Is my understanding of the current code correct?
This bug has been marked "future" because the original netscape engineer working on this is over-burdened. If you feel this is an error, that you or another known resource will be working on this bug,or if it blocks your work in some way -- please attach your concern to the bug for reconsideration.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Mass update of qa contact
QA Contact: gerardok → janc
Blocks: 35294
nominating this for beta1 because it blocks 35294 which needs to get fixed in order to speed up mail reply performance.
Keywords: mailtrack, nsbeta1
Keywords: dom1
Component: DOM Level 1 → DOM Core
QA contact Update
QA Contact: janc → desale
Nom catfood: message reply performance is about to become our biggest perf problem in mail... (Is this the active bug or is there another somewhere?)
Keywords: nsCatFood
Keywords: nsbeta1 → nsbeta1+
Target Milestone: Future → mozilla0.9.1
Duping this of bug 35294 which is really about the same problem. *** This bug has been marked as a duplicate of 35294 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
verified dups
Status: RESOLVED → VERIFIED
Component: DOM: Core → DOM: Core & HTML
QA Contact: desale → general
You need to log in before you can comment on or make changes to this bug.