this is a performnace problem. nsIPlatformCharset is design to be an singleton service object. Create it multiple time is a performance problem. Somhow currently many places called by CreateInstance, which cause multiple copy of it get create. We should replace them by NS_WITH_SERVICE

There are many places. I use this bug as tracking bug

By also fixing 30213, we can reduce a lot of unnecessary file reading.

Frank, how much performance improvement do we get from all this. When we looked only two instances were being created on startup adding to .14secs out of 19sec startup right ? I can check my quantify data if you tell me something to look for and post the timings here. Then we can decide to fix this if we think the gain is high.

I have changed most of the places to use the service manager in my local tree. And saw most of the class store the result charset into static variable, so it won't call the interface as many times as originally expected. nsFileSpec does not cache the result but the code is executed in limited condition (i.e., not used by most of the nsFileSpec instance).

this is a meta bug. Mark it as futre to make it out of the radar

It's fixed.