Closed
Bug 571148
Opened 16 years ago
Closed 16 years ago
Replace simple-storage's UninstallNotifier with new unload notifications
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
0.5
People
(Reporter: adw, Assigned: adw)
Details
Attachments
(1 file)
|
5.95 KB,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
context-menu uses its own uninstall notification mechanism so it can purge persistent storage. Bug 549324 landed and provides a Jetpacky way to listen for uninstalls, so context-menu should switch to that. (This also means that its ad hoc self.id + "@jetpack" ID construction is no longer necessary.) But see also bug 571049.
| Assignee | ||
Comment 1•16 years ago
|
||
D'oh, s/context-menu/simple-storage/g.
| Assignee | ||
Comment 2•16 years ago
|
||
Updated•16 years ago
|
Attachment #450489 -
Flags: review?(myk) → review+
Comment 3•16 years ago
|
||
Comment on attachment 450489 [details] [diff] [review]
patch
Looks great, works well. r=myk
| Assignee | ||
Comment 4•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•16 years ago
|
Target Milestone: -- → 0.5
Comment 5•15 years ago
|
||
The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.
To filter bugmail related to this change, filter on the word "looptid".
Component: Jetpack SDK → General
Product: Mozilla Labs → Add-on SDK
QA Contact: jetpack-sdk → general
Version: Trunk → unspecified
You need to log in
before you can comment on or make changes to this bug.
Description
•