Closed
Bug 53608
Opened 25 years ago
Closed 25 years ago
1% of mailnews scrolling time spent in NS_NewAtom()
Categories
(Core :: XUL, defect, P1)
Tracking
()
VERIFIED
FIXED
M18
People
(Reporter: waterson, Assigned: waterson)
Details
(Keywords: perf, Whiteboard: [nsbeta3+][nsbeta3++][PDTP1])
Attachments
(1 file)
|
4.33 KB,
patch
|
Details | Diff | Splinter Review |
...from stupid calls by nsXULContentUtils::GetElement[Ref]Resource(). Attaching
patch to use nsXULAtoms.
|
Assignee | |
Comment 1•25 years ago
|
||
|
|
Assignee | |
Comment 2•25 years ago
|
||
Marking perf, nominating as nsbeta3 with all the usual grafitti to push it
through the system. This is an extremely low-risk change, and although it won't
save the world, it's one of the thousand cuts that's killing us.
|
|
||
Comment 3•25 years ago
|
||
r=hyatt
PDT: Going out on a limb. If risk is truly minimal, mail/news scrolling is one
of our biggest usability issues. Will take for PR3. Please "guarantee" this
won't hoark the tree! If in any doubt, hold for RTM.
Whiteboard: [nsbeta3+] → [nsbeta3+][nsbeta3++][PDTP1]
|
|
Assignee | |
Comment 5•25 years ago
|
||
Checked in on tip and branch.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
|
||
Comment 6•25 years ago
|
||
checked in, trunk and branch, verified. 999 cuts to go.
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•