Closed
Bug 526012
Opened 15 years ago
Closed 14 years ago
audit observer service usage
Categories
(Firefox :: Sync, defect, P3)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
1.3b1
People
(Reporter: mconnor, Assigned: mconnor)
References
Details
Attachments
(1 file)
|
3.34 KB,
patch
|
Mardak
:
review+
|
Details | Diff | Splinter Review |
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•15 years ago
|
Priority: -- → P3
Target Milestone: --- → 1.1
|
||
Updated•14 years ago
|
Target Milestone: 1.1 → 1.2
|
|
Assignee | |
Updated•14 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•14 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•14 years ago
|
||
Attachment #440030 -
Flags: review?(edilee)
|
||
Comment 3•14 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•14 years ago
|
||
http://hg.mozilla.org/labs/weave/rev/0d5b4fae7532
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•14 years ago
|
Target Milestone: 1.3 → 1.3b1
|
|
||
Updated•14 years ago
|
|
||
Updated•6 years ago
|
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•