Closed Bug 1155989 Opened 9 years ago Closed 9 years ago

Dump the rejection of the promise returned by ServiceWorkerRegistration.unregister() in our tests

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

      No description provided.
Blocks: 1112469
Comment on attachment 8594350 [details] [diff] [review]
Dump the rejection of the promise returned by ServiceWorkerRegistration.unregister() in our tests

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

::: dom/workers/test/serviceworkers/test_close.html
@@ +43,3 @@
>              SimpleTest.finish();
> +          }, function(e) {
> +            dump("Unregistering the SW failed with " + e + "\n");

SimpleTest.finish()

::: dom/workers/test/serviceworkers/test_empty_serviceworker.html
@@ +26,5 @@
>      registration.unregister().then(function(success) {
>        ok(success, "unregister worked");
>        SimpleTest.finish();
> +    }, function(e) {
> +      dump("Unregistering the SW failed with " + e + "\n");

finish here too.

::: dom/workers/test/serviceworkers/test_serviceworker_interfaces.html
@@ +20,5 @@
>            ok(success, "The service worker should be unregistered successfully");
>  
>            SimpleTest.finish();
> +        }, function(e) {
> +          dump("Unregistering the SW failed with " + e + "\n");

and here

::: dom/workers/test/serviceworkers/test_serviceworker_not_sharedworker.html
@@ +41,5 @@
>            registration.unregister().then(function(success) {
>              ok(success, "unregister should succeed");
>              SimpleTest.finish();
> +          }, function(e) {
> +            dump("Unregistering the SW failed with " + e + "\n");

and here.
Attachment #8594350 - Flags: review?(nsm.nikhil) → review+
https://hg.mozilla.org/mozilla-central/rev/df337fdbcc2d
Assignee: nobody → ehsan
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: