Closed Bug 588249 Opened 14 years ago Closed 11 years ago

Make AddonRepository smarter about what repository Addons it keeps in memory

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: Unfocused, Assigned: Unfocused)

References

Details

Attachments

(1 file)

With bug 586591, AddonRepository caches addon data in a SQLite database, and also in memory. However, it doesn't need to keep all addons in memory - and should be smart about what can just be re-retrieved from the database.
Attached patch WiP v1 (deleted) — Splinter Review
Work in progress. Has test failures. Will try to finish it off after beta 5.
Assignee: nobody → bmcbride
Comment on attachment 466884 [details] [diff] [review] WiP v1 Oh, and this still holds on to all addon objects forever, once it gets them. Need to add in some magic pixie dust to make that not happen. It does, however, retrieve them lazily.
Has bug 853389 made this invalid?
Flags: needinfo?(bmcbride)
Oh, yes, indeed.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(bmcbride)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: