Closed Bug 794563 Opened 7 years ago Closed 7 years ago
Make app uninstall trigger new clear private data notifications
+++ This bug was initially created as a clone of Bug #784378 +++ Need to hook these two together
Note: We'll want to remove most or all the uninstall listeners.
This does the job.
Attachment #665671 - Flags: review?(mounir)
Attachment #665671 - Flags: review+
Attachment #665671 - Flags: review?(mounir) → review+
Also fixed a QI missing from WebApps.jsm: https://hg.mozilla.org/mozilla-central/rev/cdd4506bc66c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
This looks testable from the DOM level, but want to confirm. I'm wondering if we can get an automated test here as well. From an end-user perspective, say I uninstall the app from the homescreen that I previously installed. What should I watch out for in this case in relation to this implementation?
QA Contact: jsmith
Whiteboard: [LOE:M][WebAPI:P1] → [LOE:M], [WebAPI:P1], [qa?]
This isn't really testable from a user perspective. However, a test should be written to make sure the event is sent. Ben, do you think you can write the test?
Whiteboard: [LOE:M], [WebAPI:P1], [qa?] → [LOE:M], [WebAPI:P1]
Whiteboard: [LOE:M], [WebAPI:P1] → [LOE:M], [WebAPI:P1], [qa-]
You need to log in before you can comment on or make changes to this bug.