Closed Bug 584369 Opened 14 years ago Closed 14 years ago

history sync is busted on tip

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mconnor, Assigned: mconnor)

References

Details

Attachments

(1 file)

Attached patch v1Splinter Review
Observers isn't directly defined.  Not sure how this got missed, or how it addressed the leaks, but the fix is easy.
Attachment #462780 - Flags: review?(philipp)
Comment on attachment 462780 [details] [diff] [review]
v1

We really need some history test coverage. At least bookmarks and form have *some* tests.
Attachment #462780 - Flags: review?(philipp) → review+
http://hg.mozilla.org/services/fx-sync/rev/a339c14aa8e7
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: