97182 - AutoConfig is created twice as part of netscape.cfg reading

Status: RESOLVED FIXED

(Core :: Preferences: Backend, defect)

Description

[Mitesh Shah]

Right now, autoconfig object is created inside readconfig() in nsPrefSerice.cpp.
As a result of a fix for bug 92447, the readconfig will be called again when
resetprefs() happens in profile migration case or runtime profile switching (in
the future). We need to be able to release autoconfig and restart autoconfig
object with the new values which is not the case right now.

Comment 1

[lrg]

*** Bug 87661 has been marked as a duplicate of this bug. ***

Comment 2

[lrg]

*** Bug 86688 has been marked as a duplicate of this bug. ***

Comment 3

[Mitesh Shah]

How abt we use do_GetService instead do_CreateInstance. I think that will make
sure that the AutoConfig is created as a singleton object.

Comment 4

[tao]

mitesh is no longer with us. reassigning to default module owner. bug 113387
tracks the original list of mitesh bugs

Comment 5

[Brian Nesse (gone)]

This will be fixed by the patch in bug 89137.

Comment 6

[Brian Nesse (gone)]

Patch for bug 89137 has landed. This is now fixed.

Comment 7

[sairuh (rarely reading bugmail)]

rodney, luke, would you like to verify this one? thx!

Comment 8

[lrg]

Brian, i am not quite sure on how to go about testing this, since the extra
process creation is pretty invisible.  Any suggestions?

Comment 9

[Brian Nesse (gone)]

It's really a code based thing. You won't be able to verify it in any visually
obvious way. I guess I could show you the code which keeps it from creating
multiple instances...