1462467 - Bring the remoted registration up-to-date when it attaches to the parent-side ServiceWorkerRegistrationInfo

Status: RESOLVED INVALID

(Core :: DOM: Service Workers, enhancement, P2)

Description

[Ben Kelly [:bkelly, not reviewing]]

Testing shows that we need to handle a couple conditions when a ServiceWorkerRegistration attaches to a parent-side registration info:

1. If the registration just started installing then we need to fire an updatefound event.  The parent SWM does try to do this, but the actor for the registration does not exist yet, so it missed it.
2. Update the registration's state if the SW has actually moved to waiting/active by the time the registration's actor attaches.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Attachment: Auto-update the ServiceWorkerRegistration's state when it attaches to the parent-side registration info. r=baku

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Attachment: Auto-update the ServiceWorkerRegistration's state when it attaches to the parent-side registration info. r=baku

Auto-update the state of the registration when it attaches to the parent SWM.  Please see the code comment for a more complete description of why this needs to happen and why its safe per the spec.

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

Sorry, I meant to drop the flag on this patch.  I intend to fold this together with some other changes.  This patch was not really adequate by itself.

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

The patch here was not adequate by itself and is overcome by other changes.  Just going to jetison this bug in favor of the others I have open.