This was going to be checked in with bug 86021 but that's no longer going into the branch. This needs to for the sake of embedding clients using profile switching. The problem is this: nsISingleSignon observes profile changes. Since these are components and not a service, if the last instance is destroyed before switching the profile (typical) there's nothing to observe the change and signon data is not flushed. I'm not sure why this was never noticed before. Maybe we were leaking a signon.
we should use app-startup components to instantiate a profile change listener...
Alec, this is the same patch that you sr'd before with changes from your comments. The static member var is no longer an nsIObserver* but a simple PR_Bool and it's commented a bit better. Can you sr= this one? Steve, can you r=?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.3
Alec, I didn't see your comment before posting that patch. > we should use app-startup components I like that idea. On the other hand, this patch is tested and ready.
works for me! sr=alecf
Sorry, Steve's on vacation this week. But I approve your making this change to the wallet module. But you'll need to get someone else to do the review if you want it done this week.
let's get this into the branch and trunk.
Fix checked into branch & trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Verified fix checked into lxr.mozilla.org
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.