Closed Bug 96681 Opened 23 years ago Closed 9 years ago

delay calling of nsHttpHandler::InitUserAgentComponents()

Categories

(Core :: Networking: HTTP, defect, P3)

defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: ftang, Unassigned)

References

Details

(Keywords: perf)

Attachments

(1 file)

I found this issue while I try to address 96678. the nsHttpHandler::InitUserAgentComponents() load nslocale.dll indrectly while it callt the nsPref::GetLocalizedUnicharPref I look at the stack, it seems the reason the protocol handler get create is because the nsCodebasePrincipal::InitFromPersistent Do we need to create useragentcomponents in that particulare moment? can we delay it untill we really need it ? If we can, then we may speed up startup performence by not loading nslocale.dll (with other changes)
Keywords: perf
Blocks: 75041
No longer blocks: 75041
-> darin
Assignee: neeti → darin
this should be easy to do.
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.5
Blocks: 75041
Blocks: 96678
Attached patch v1.0 fixes the problem (deleted) — Splinter Review
looks like delaying InitUserAgentComponents may not be enough... nsHttpHandler::Init() also calls PrefsChanged(), which ends up calling GetLocalizedUnicharPref(). it'll be a bit more work to delay the code in PrefsChanged().
-> untargeted, since i'm not sure we really want to do this.
Target Milestone: mozilla0.9.5 → ---
-> future, perhaps WONTFIX
Target Milestone: --- → Future
Assignee: darin → nobody
Status: ASSIGNED → NEW
QA Contact: tever → networking.http
Target Milestone: Future → ---
Is this WONTFIX?
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: