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

Status: RESOLVED FIXED

(Core :: XPCOM, defect, P2)

Description

[Suresh Duddi (gone)]

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

Comment 1

[Suresh Duddi (gone)]

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.

Comment 2

[Suresh Duddi (gone)]

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.

Comment 3

[Warren Harris]

Edward: Welcome to xpcom!

Comment 4

[Warren Harris]

Once again... attempting to reassign from Ray to Edward.

Comment 5

[Blake Ross]

Ed, do you know the easiest way to fix these usages (I'm guessing one exists 
since Suresh said he eliminated 3,000)?

Comment 6

[Edward Kandrot]

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?

Comment 7

[Doug Turner (:dougt)]

reassign all kandrot xpcom bug.

Comment 8

[Doug Turner (:dougt)]

this has been fixed by my merging of the component and service manager.