Closed Bug 49786 Opened 20 years ago Closed 19 years ago

5000 calls to ProgIDToClassID() for simple startup/shutdown test. Clients need to cache progid.

Categories

(Core :: XPCOM, defect, P2)

x86
All
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: dp, Assigned: dougt)

References

Details

(Keywords: perf, Whiteboard: [nsbeta3-][nav+perf])

This costs in terms of performance bloat and more work not only at startup but
over the long run like while viewing/scrolling messages in mail thread pane.

Succeeded ProgIDToClassID() : 4976
  961 component://netscape/xbl
  505 nsThreadJSContextStack
  491 component://netscape/intl/unicode/encoder?charset=ISO-8859-1
  434 component://netscape/network/protocol?name=file
  401 component://netscape/network/protocol?name=chrome
  373 component://netscape/network/protocol?name=resource
  300 component://netscape/observer-service
  232 component://netscape/scriptsecuritymanager
  149 component://netscape/timer
  113 component://netscape/filelocator
  102 component://netscape/intl/unicode/decoder?charset=UTF-8
   84 component://netscape/rdf/container
   78 component://netscape/uriloader
   77 component://netscape/rdf/xul-content-utils
   77 component://netscape/rdf/xul-prototype-cache
   74 component://netscape/network/standard-url
   74 component://netscape/preferences
   58 layout.content-policy.1
   54 component://netscape/intl/unicode/decoder?charset=ISO-8859-1
Nominating for beta3. A side effect of this is that such analysis reveals a lot
of information.

We will get whatever we can fix with reduced risk.
Status: NEW → ASSIGNED
Keywords: nsbeta3, perf
OS: Linux → All
Priority: P3 → P2
Whiteboard: [nsbeta3+] until 9/14
Target Milestone: --- → M20
beta3-. Impact isnt huge. I fixed about 3000 of these. But quantify sez it all
0.02 secs altogether. Would be nice to fix them for later.
Assignee: dp → rayw
Status: ASSIGNED → NEW
Whiteboard: [nsbeta3+] until 9/14 → [nsbeta3-]
Target Milestone: M20 → Future
Status: NEW → ASSIGNED
Edward: Welcome to xpcom!
Status: ASSIGNED → NEW
QA Contact: leger → rayw
Target Milestone: Future → mozilla1.0
Once again... attempting to reassign from Ray to Edward.
Assignee: rayw → kandrot
Status: NEW → ASSIGNED
Blocks: 7251
Ed, do you know the easiest way to fix these usages (I'm guessing one exists 
since Suresh said he eliminated 3,000)?
Whiteboard: [nsbeta3-] → [nsbeta3-][nav+perf]
I do not know if there is a way to fix this, since I could not find
ProgIDToClassID in lxr or by doing a grep in XPCOM.  Is there a different name
for this function or has it gone away?
No longer blocks: 7251
Blocks: 7251
reassign all kandrot xpcom bug.
Assignee: kandrot → dougt
Status: ASSIGNED → NEW
Target Milestone: mozilla1.0 → ---
Blocks: 98275
this has been fixed by my merging of the component and service manager.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.