avoid "up crawling" every time CreateTemplateContents() is called

RESOLVED FIXED in Future

Status

()

Core
RDF
P3
normal
RESOLVED FIXED
18 years ago
15 years ago

People

(Reporter: Chris Waterson, Assigned: Chris Waterson)

Tracking

({perf})

Trunk
Future
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

18 years ago
The XUL template builder currently does a *ton* of "up crawling" (which involves
attribute fetching and string copying) in CreateTemplateContents(). This is the
routine that is called to "flesh out" a partially instantiated template; e.g.,
when you scroll around in the mail window and hit a message that you've never
seen before.

I'm not sure exactly how we could fix this; maybe using another lookaside table
(or merging it with the conflict set, or something), but it does account for a
couple percentage points of the profile when scrolling in mailnews. (Most of the
time goes to looking for a 'ref' or 'id' attribute, and then calling
rdf_MakeAbsoluteURI() -- all string horror.)
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Keywords: perf
Target Milestone: --- → Future
(Assignee)

Comment 1

17 years ago
Fixed in patch to bug 71530.
Depends on: 71530

Comment 2

17 years ago
if this is fixed, shouldn't it be marked as such to remove it from any radar? 
I sadly don't have the power to do this from this account...
(Assignee)

Comment 3

17 years ago
Yes, thanks. I must've forgot to mark it as such.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Keywords: donttest
Resolution: --- → FIXED

Comment 4

15 years ago
tever is not RDF QA anymore
QA Contact: tever → nobody
You need to log in before you can comment on or make changes to this bug.