1219205 - ServiceWorkerInfo should be an XPCOM object.

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, defect)

Description

[Eddy Bruel [:ejpbruel]]

Currently, nsIServiceWorkerRegistrationInfo does not provide access to its service workers. Each of these service workers is represented internally by an instance of ServiceWorkerInfo.

We would like to turn these instances of ServiceWorkerInfo into XPCOM objects, so we can expose them as properties on nsIServiceWorkerRegistrationInfo. Once we have  obtained a reference to an nsIServiceWorkerInfo from an nsIServiceWorkerRegistrationInfo, we will be able to use the former to attach a debugger to a particular service worker.

Comment 1

[Eddy Bruel [:ejpbruel]]

Attachment: ServiceWorkerInfo should be an XPCOM object.

Note that with this change we no longer need properties like currentWorkerURL or activeCacheName, since these can be reimplemented as properties on individual ServiceWorkerInfos.

Comment 2

[Cătălin Badea (:catalinb)]

(In reply to Eddy Bruel [:ejpbruel] from comment #0)
> Currently, nsIServiceWorkerRegistrationInfo does not provide access to its
> service workers. Each of these service workers is represented internally by
> an instance of ServiceWorkerInfo.
> 
> We would like to turn these instances of ServiceWorkerInfo into XPCOM
> objects, so we can expose them as properties on
> nsIServiceWorkerRegistrationInfo. Once we have  obtained a reference to an
> nsIServiceWorkerInfo from an nsIServiceWorkerRegistrationInfo, we will be
> able to use the former to attach a debugger to a particular service worker.

One important thing to note is that we need to attach to the ServiceWorkerInfo
living in the content process.

Comment 3

[Eddy Bruel [:ejpbruel]]

Try run for this patch:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fc75e55dd66a

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9d8485c0d32a

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9d8485c0d32a