Replace simple-storage's UninstallNotifier with new unload notifications

RESOLVED FIXED in 0.5

Status

RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: adw, Assigned: adw)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
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

9 years ago
D'oh, s/context-menu/simple-storage/g.
(Assignee)

Comment 2

9 years ago
Created attachment 450489 [details] [diff] [review]
patch
Assignee: nobody → adw
Status: NEW → ASSIGNED
Attachment #450489 - Flags: review?(myk)
Attachment #450489 - Flags: review?(myk) → review+
Comment on attachment 450489 [details] [diff] [review]
patch

Looks great, works well. r=myk
(Assignee)

Comment 4

9 years ago
http://hg.mozilla.org/labs/jetpack-sdk/rev/3d32b233922d
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Updated

9 years ago
Target Milestone: -- → 0.5
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.