HTTP should provide a "first-time-needed" category for delayed loading.

VERIFIED INVALID

Status

()

P3
normal
VERIFIED INVALID
19 years ago
18 years ago

People

(Reporter: jud, Assigned: gagan)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
We want to delay loading of header observers in HTTP (nsINetModuleMgr). I think
this should work by loading the category (which header observers register
under)  in HTTP on the first HTTP request. Trying to do header level loading
seems heavy, though I guess we could create a category for each header :-/.
(Reporter)

Comment 1

19 years ago
fixing spelling in summary
Summary: HTTP should provice a "first-time-needed" category for delayed loading. → HTTP should provide a "first-time-needed" category for delayed loading.
(Assignee)

Comment 2

19 years ago
why?
(Reporter)

Comment 3

19 years ago
to delay loading of HTTP specific modules in an attempt to speed up startup.
(Assignee)

Comment 4

19 years ago
I got that part, but the HTTP handler itself doesn't load till the first http
request. We call the HTTP startup category only after that, and then it doesn't
save us much since the first request is the one that loaded the handler in the
first place. I would let DP say more if there is a way to delay other things to
load which register for a category...
Assignee: gagan → dp

Comment 5

19 years ago
We already have this. Http on first request will create instances of 
CID/Progid registered under category "http-startup-category"

I dont think this will improve startup but certainly will eliminate 
depedencies.

Comment 6

19 years ago
Back to gagan. Let me know if you want me help me fit in any other component
into this category approach.
Assignee: dp → gagan
(Assignee)

Comment 7

19 years ago
Based on the discussions above this seems invalid. marking as such. Reopen if 
you have a better suggestion. 
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → INVALID

Comment 8

18 years ago
verified invalid
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.