Closed Bug 794563 Opened 7 years ago Closed 7 years ago

Make app uninstall trigger new clear private data notifications

Categories

(Core Graveyard :: DOM: Apps, defect)

defect
Not set

Tracking

(blocking-basecamp:+)

RESOLVED FIXED
blocking-basecamp +

People

(Reporter: bent.mozilla, Assigned: bent.mozilla)

References

(Blocks 1 open bug)

Details

(Keywords: feature, Whiteboard: [LOE:M], [WebAPI:P1], [qa-])

Attachments

(1 file)

+++ 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.
Assignee: nobody → bent.mozilla
Blocks: 795134
Attached patch Patch, v1Splinter Review
This does the job.
Attachment #665671 - Flags: review?(mounir)
Blocks: 795203
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
Component: DOM: Mozilla Extensions → DOM: Apps
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?
Flags: in-testsuite?
Whiteboard: [LOE:M], [WebAPI:P1], [qa?] → [LOE:M], [WebAPI:P1]
Whiteboard: [LOE:M], [WebAPI:P1] → [LOE:M], [WebAPI:P1], [qa-]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.