We do either Observers.js or straight XPCOM, and it's kinda confusing. If the wrapper is worth it, let's do that everywhere, or drop it entirely.
Priority: -- → P3
Target Milestone: --- → 1.1
Assignee: nobody → mconnor
OS: Mac OS X → All
Hardware: x86 → All
Summary: clean up observer service usage → audit observer service usage
Target Milestone: 1.2 → 1.3
Okay, so Observers.js is a little saner/simpler, and gives us some added flexibility, so I think it's easier to just use that consistently... patch upcoming.
Status: NEW → ASSIGNED
Created attachment 440030 [details] [diff] [review] kill svc.observers, use svc.obs everywhere
Attachment #440030 - Flags: review?(edilee)
Comment on attachment 440030 [details] [diff] [review] kill svc.observers, use svc.obs everywhere Some reason I remember there being more... but maybe they gradually migrated to Svc.Obs already.
Attachment #440030 - Flags: review?(edilee) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.