794563 - Make app uninstall trigger new clear private data notifications

Status: RESOLVED FIXED

(Core Graveyard :: DOM: Apps, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

+++ This bug was initially created as a clone of Bug #784378 +++

Need to hook these two together

Comment 1

[Justin Lebar (not reading bugmail)]

Note: We'll want to remove most or all the uninstall listeners.

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch, v1

This does the job.

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Also fixed a QI missing from WebApps.jsm:

https://hg.mozilla.org/mozilla-central/rev/cdd4506bc66c

Comment 4

[Jason Smith [:jsmith]]

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?

Comment 5

[Mounir Lamouri (:mounir)]

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?