1155988 - Use the ready promise more often in service worker tests

Status: RESOLVED FIXED

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

Comment 1

[(no longer active)]

Attachment: Use the ready promise more often in service worker tests

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8594348 [details] [diff] [review]
Use the ready promise more often in service worker tests

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

::: dom/workers/test/serviceworkers/fetch/context/register.html
@@ +7,2 @@
>    function done(reg) {
> +    ok(reg.waiting || reg.active, "Either active or waiting worker should be available.");

if ready resolved, only reg.active should be non-null in this case.

::: dom/workers/test/serviceworkers/fetch/https/clonedresponse/register.html
@@ +7,2 @@
>    function done(reg) {
> +    ok(reg.waiting || reg.active, "Either active or waiting worker should be available.");

ditto.

::: dom/workers/test/serviceworkers/fetch/https/register.html
@@ +7,2 @@
>    function done(reg) {
> +    ok(reg.waiting || reg.active, "Either active or waiting worker should be available.");

here too

::: dom/workers/test/serviceworkers/fetch/sandbox/register.html
@@ +7,2 @@
>    function done(reg) {
> +    ok(reg.waiting || reg.active, "Either active or waiting worker should be available.");

same here.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fe30eb26b6ab

Comment 4

[Carsten Book [:Tomcat]]

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