venkman registers an app start observer, to allow the user to set a pref indicating that the debugger should be initialized at startup. It does not create an instance of the service unless the pref is set (see js/jsd/jsd_xpc.cpp.) XPCOM is free to unload the service, if it does that kind of thing. If this is an embedding issue, then maybe the embedding builds should turn off venkman.
No, this isn't an embedding issue per se. I just noticed that it was loaded, and filed the bug since, in principle, any dll that does not need to be loaded during startup should be deferred. If it needs to be loaded, then it needs to be loaded. If there is some other way to refactor this to avoid loading the DLL though, then maybe that could be considered. At any rate, I leave it to the pros to figure out what to do with this (i.e., what is the right balance).
Frankly, the feature I'm trying to provide is currently broken. If loading the debugger is a real startup perf hit, I could try figure a compromise between performance and functionality. If it's something that causes a large hit NOW, I'm ok with backing out the startup listener, which would prevent the service from being loaded at startup.
Yes please remove it from being loaded on startup. We have been on a crusade to eliminate dlls that get loaded on startup. 1/2 the startup time is spent loading dlls.
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Comment on attachment 52158 [details] [diff] [review] patch removes app start listener from debugger service sr=shaver
Attachment #52158 - Flags: superreview+
Comment on attachment 52158 [details] [diff] [review] patch removes app start listener from debugger service r=dp
Attachment #52158 - Flags: review+
Comment on attachment 52158 [details] [diff] [review] patch removes app start listener from debugger service a=asa (on behalf of drivers) for checkin to 0.9.5.
Attachment #52158 - Flags: approval+
checked in to branch and trunk
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Verify fixed. no more loading on startup.
Status: RESOLVED → VERIFIED
QA Contact: rginda → caillon
You need to log in before you can comment on or make changes to this bug.