js debugger [jsd3250.dll] is loaded on startup (win32)

VERIFIED FIXED

Status

defect
VERIFIED FIXED
18 years ago
8 months ago

People

(Reporter: jrgmorrison, Assigned: rginda)

Tracking

({perf})

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

18 years ago
I noticed that with the Javascript debugger now built by default, we are 
loading its DLL (jsd3250.dll) at startup. Seems like the sort of thing that 
should happen lazily, since (despite the wonderful goodness that this is) this 
is a feature that only some subset of all users will ever use the debugger.
Assignee

Comment 1

18 years ago
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.
Reporter

Comment 2

18 years ago
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).

Updated

18 years ago
Blocks: 7251
Keywords: perf
Assignee

Comment 3

18 years ago
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.
Assignee

Comment 6

18 years ago
s/sr anyone?
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 9

18 years ago
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+
Assignee

Comment 10

18 years ago
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
Product: Core → Other Applications

Updated

8 months ago
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.