audit observer service usage

RESOLVED FIXED in 1.3b1

Status

Cloud Services
Firefox Sync: Backend
P3
normal
RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: mconnor, Assigned: mconnor)

Tracking

unspecified
1.3b1
Points:
---
Bug Flags:
blocking-weave1.3 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
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.
(Assignee)

Updated

9 years ago
Priority: -- → P3
Target Milestone: --- → 1.1

Updated

9 years ago
Target Milestone: 1.1 → 1.2
(Assignee)

Updated

8 years ago
Assignee: nobody → mconnor
Flags: blocking-weave1.3+
OS: Mac OS X → All
Hardware: x86 → All
Summary: clean up observer service usage → audit observer service usage
Target Milestone: 1.2 → 1.3
(Assignee)

Comment 1

8 years ago
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
(Assignee)

Comment 2

8 years ago
Created attachment 440030 [details] [diff] [review]
kill svc.observers, use svc.obs everywhere
Attachment #440030 - Flags: review?(edilee)

Comment 3

8 years ago
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+
(Assignee)

Comment 4

8 years ago
http://hg.mozilla.org/labs/weave/rev/0d5b4fae7532
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Updated

8 years ago
Target Milestone: 1.3 → 1.3b1

Updated

8 years ago
Depends on: 568292

Updated

8 years ago
Blocks: 568292
No longer depends on: 568292
You need to log in before you can comment on or make changes to this bug.