Add a test to check that cache storage is cleaned up when uninstalling an app

RESOLVED FIXED in Firefox 42

Status

()

defect
RESOLVED FIXED
4 years ago
3 months ago

People

(Reporter: ferjm, Assigned: ferjm)

Tracking

unspecified
mozilla42
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

Apart from unregistering the service workers (Bug 1169249) we need to clear the associated caches when uninstalling apps.
Assignee: nobody → ferjmoreno
Whiteboard: [s4]
Target Milestone: --- → NGA S3 (26Jun)
I think you want to clear the entire app origin, not just the cache.  You can do this with QuotaManager.
Summary: Clear service worker cache when uninstalling its owner app → Clear QuotaManager storage when uninstalling an app
Status: NEW → ASSIGNED
Whiteboard: [s4]

Updated

4 years ago
Target Milestone: NGA S3 (26Jun) → FxOS-S1 (26Jun)
Summary: Clear QuotaManager storage when uninstalling an app → Add a test to check that cache storage is cleaned up when uninstalling an app
Posted patch v1Splinter Review
It seems that we get this for free by using the QuotaManager on the Cache API [1]. I only added a test for this case.

[1] https://mxr.mozilla.org/mozilla-central/source/dom/quota/QuotaManager.cpp#3044
Attachment #8627131 - Flags: review?(bkelly)
Target Milestone: FxOS-S1 (26Jun) → FxOS-S2 (10Jul)
Attachment #8627131 - Flags: review?(bkelly) → review+
Target Milestone: FxOS-S2 (10Jul) → FxOS-S1 (26Jun)
Target Milestone: FxOS-S1 (26Jun) → FxOS-S2 (10Jul)
Target Milestone: FxOS-S2 (10Jul) → FxOS-S1 (26Jun)
sorry had to back this out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=11207175&repo=mozilla-inbound
Flags: needinfo?(ferjmoreno)
Target Milestone: FxOS-S1 (26Jun) → ---
https://hg.mozilla.org/mozilla-central/rev/51dc428a5250
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.