97462 - Delay cookie dll initialization even further

Status: RESOLVED WORKSFORME

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

Description

[Suresh Duddi (gone)]

nsHttpHandler get intialized early due to nsCodebasePrincipal doing a
NS_NewURI() This causes Cookie dll to be loaded early at nsHttpHandler::Init()
via Categories

Comment 1

[Suresh Duddi (gone)]

Attachment: Delay issuing http-startup notification until the first nsHttpRequest is created.

Comment 2

[Suresh Duddi (gone)]

Darin, I checked this patch. There are two clients of the httpstartup category:
nsRegistry::Common -  software -  mozilla -  XPCOM -  categories - 
http-startup-category
    Http Cookie Notify	 = @mozilla.org/cookie-notifier;1
    Content UserAgent Setter	 = @mozilla.org/content/http-startup;1
Both are fine with this delayed.

Comment 3

[Darin Fisher]

dp: i'm not sure making http initialization lazy is the right thing to do.  the
only reason http is being initialized at all is because of the UniversalFoo
hack.  i talked to mstolz and he said that an http url is being created as a
hack, that somehow fixed a java crash on linux.  no one seems to understand why
this was necessary.  but, if we solve that problem, then i think all of these
bugs related to making http initialization lazy would go away.

Comment 4

[Darin Fisher]

dp: just remove these two lines from all.js:

pref("capability.principal.codebase.foo.id", "http://www.netscape.com");
pref("capability.principal.codebase.foo.granted", "UniversalFoo");

Comment 5

[Suresh Duddi (gone)]

The reason this hack is there sounds strange though. Maybe java has changed
enough and it is time to see if the hack is even required.

Definitely sounds like the right solution. Totally second that.

Comment 6

[Blake Ross]

(The Java fiasco was recent, so it's unlikely, but worth a shot)

Comment 7

[Cathleen]

I think java crash bug is bug 89488, targeted for 0.9.6.
Maybe we can have this fixed by 0.9.7?

Comment 8

[Darin Fisher]

would be really nice to sort out the universal foo crap for moz 1.0

Comment 9

[Darin Fisher]

-> dougt

Comment 10

[Doug Turner (:dougt)]

marking dup.

*** This bug has been marked as a duplicate of 89488 ***

Comment 11

[John Unruh]

Verified duplicate.

Comment 12

[benc]

Dupes should not be depends the same bug.

Comment 13

[Doug Turner (:dougt)]

mass reassigning to nobody.