Created attachment 645038 [details] Fix v1 - Add interface methods and test Right now to observe operations on all calendars at once you have to: * Create a composite calendar, add in all current calendars * Observe the calendar manager to reflect additions/deletions into the composite calendar * Add an observer on the composite calendar I'd like to simplify this a bit so all you need to do is: * Add a (new type of) observer on the calendar manager. This patch takes care and also adds a test. If you are wondering about the large removal of getPrefBranchFor and friends, these functions are just moved (without change) to the bottom of the file so that the actual calendar manager is at the top.
Comment on attachment 645038 [details] Fix v1 - Add interface methods and test Actually, there is only this part. I'm so used to missing this in extensions I create that I wasn't aware this isn't used in core (yet).
Comment on attachment 645038 [details] Fix v1 - Add interface methods and test Looks good. r=mmecca
Pushed to comm-central changeset 71eae60afbf6