60583 - Cyclic Service Dependency between nsHTTPHandler::InitUserAgentComponents() and nsLayoutModule::SetUserAgent()

Status: VERIFIED DUPLICATE of bug 46488

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

Description

[Patrick C. Beard]

nsHTTPHandler::InitUserAgentComponents() is called when the HTTP protocol handler 
is first created via the service manager. At the URL above is a line of that 
method that calls mPrefs->GetLocalizedUnicharPref(UA_PREF_PREFIX "locale", which 
eventually reaches nsLayoutModule::SetUserAgent() (see attached stack crawl to 
see how that happens). nsLayoutModule::SetUserAgent() in turn tries to access the 
service signified by kHTTPHandlerCID, which of course is nsHTTPHandler again. I 
don't know how this circularity creeped in, nor how it should be fixed.

One clue:  I'm seeing this when running the Mac PowerPlant embedding client.

Comment 1

[Patrick C. Beard]

Attachment: stack crawl showing cyclic service dependency

Comment 2

[Conrad Carlen (not reading bugmail)]

The question is: why doesn't this happen with SeaMonkey? After all, what
initiates this is a simple URL load. It may be that
GetDefaultLocalizedUnicharPref is called earlier in SeaMonkey, chrome is
initialized earlier, or something. The circular dependency is there but is being
masked somehow. I'll check it out. 

Comment 3

[Judson Valeski]

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

Comment 4

[benc]

VERIFIED, fixed by Layout.