Closed
Bug 80147
Opened 24 years ago
Closed 23 years ago
[XBL] Use XUL prototype elements for all XUL elements in an XBL document
Categories
(Core :: XBL, defect)
Tracking
()
RESOLVED
FIXED
mozilla0.9.6
People
(Reporter: hyatt, Assigned: hyatt)
References
(Blocks 1 open bug)
Details
(Keywords: memory-footprint, perf)
Attachments
(1 file)
|
(deleted),
patch
|
dbaron
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
If XUL elements in an XBL file used prototypes, then the act of doing CloneNode
inside GenerateAnonymousContent would become much cheaper.
|
||
Comment 1•24 years ago
|
||
I don't expect to fix this in 0.9.1, but it's one of my top priorities for
overal performance improvements. so focusing on it now.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.1
|
||
Comment 2•24 years ago
|
||
moving to 0.9.2 since this will be aimed at 0.9.2
Target Milestone: mozilla0.9.1 → mozilla0.9.2
|
|
||
Comment 3•23 years ago
|
||
any ideas on what kind of improvement we expect to see this make?
What gets affected by this change in mail? Is this still PDT+?
Keywords: perf
|
|
||
Updated•23 years ago
|
Whiteboard: [PDT+]
|
Assignee | |
Comment 8•23 years ago
|
||
Going to take this back since I see it slipping and there hasn't been much activity.
Assignee: sspitzer → hyatt
Status: ASSIGNED → NEW
|
|
||
Comment 9•23 years ago
|
||
hyatt, thanks for taking it back. Sorry for never getting to it.
You're clearly a better owner for this bug.
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9.4 → mozilla0.9.5
|
|
Assignee | |
Updated•23 years ago
|
Target Milestone: mozilla0.9.5 → mozilla0.9.6
|
|
Assignee | |
Updated•23 years ago
|
Target Milestone: mozilla0.9.6 → mozilla0.9.7
|
|
Assignee | |
Comment 10•23 years ago
|
||
Cruelly foisting this onto shaver.
Assignee: hyatt → shaver
Status: ASSIGNED → NEW
|
|
Assignee | |
Comment 11•23 years ago
|
||
Insidiously retargeting for shaver.
Target Milestone: mozilla0.9.7 → mozilla0.9.6
|
||
Updated•23 years ago
|
|
||
Comment 12•23 years ago
|
||
10/25 Perf Meeting:
hyatt is taking this bug back, due to the fact that he is working on bug 106153
Assignee: shaver → hyatt
|
|
Assignee | |
Comment 13•23 years ago
|
||
|
||
Comment 14•23 years ago
|
||
Comment on attachment 56389 [details] [diff] [review]
Here it is!
sr=jst (with some minor changes hyatt already made).
Attachment #56389 -
Flags: superreview+
|
||
Comment 15•23 years ago
|
||
Comment on attachment 56389 [details] [diff] [review]
Here it is!
r=dbaron (with changes discussed on IRC)
Attachment #56389 -
Flags: review+
|
|
Assignee | |
Comment 16•23 years ago
|
||
Fixed.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•