Make it hard to do the wrong thing with PreferenceManager sharedInstance

NEW

Status

8 years ago
8 years ago

People

(Reporter: stuart.morgan+bugzilla, Assigned: stuart.morgan+bugzilla)

Tracking

Trunk
x86
Mac OS X

Details

(Assignee)

Description

8 years ago
Given bug 644057 and all its previous friends and relations, we should be doing something smarter. Specifically, there are only one or two places where it makes sense to be creating a pref manager, given that we make one early on in startup.

What we should do, to prevent repeatedly shooting ourselves in the feet, is have sharedInstance *not* create a new one, and have an explicit call to populate it (createSharedInstance or something).

The only tricky part will be ensuring that we don't have places we currently need prefs before ensureGeckoInitted; given the complexity of the startup code (since there are at least two or three ways for us to be launched), it's likely that we do at the moment.
(Assignee)

Updated

8 years ago
Assignee: nobody → stuart.morgan+bugzilla
You need to log in before you can comment on or make changes to this bug.