Add an "Unregister" button to about:serviceworkers

RESOLVED FIXED in Firefox 40

Status

()

Core
DOM
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Ehsan, Assigned: baku)

Tracking

unspecified
mozilla40
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

3 years ago
Assignee: nobody → amarchesini
(Assignee)

Comment 1

3 years ago
Created attachment 8592860 [details] [diff] [review]
unregister.patch
Attachment #8592860 - Flags: review?(ehsan)
(Reporter)

Comment 2

3 years ago
Comment on attachment 8592860 [details] [diff] [review]
unregister.patch

Review of attachment 8592860 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/content/aboutServiceWorkers.js
@@ +127,5 @@
> +        }
> +      },
> +
> +      unregisterFailed: function() {
> +        alert(bundle.GetStringFromName('unregisterError'));

Alerting this kind of sucks, but I guess that's fine for now.
Attachment #8592860 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/60f160721580
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Depends on: 1155064
You need to log in before you can comment on or make changes to this bug.