1350433 - Don't call WorkerPrivate::WorkerName() to get service worker scope

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, enhancement)

Description

[Ben Kelly [:bkelly, not reviewing]]

In bug 1293277 I want to change how we store the scope on a WorkerPrivate.  Today we do this by putting it in WorkerName().  That is ok, but a bit of a hack.  To make bug 1293277 easier I'd like to expose a separate ServiceWorkerScope() getter.  Today it will just look at mWorkerName.  In bug 1293277 I will make it look at my new thing.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Attachment: Expose WorkerPrivate::ServiceWorkerScope() instead of using WorkerName() all over the place. r=asuth

This just makes existing code use a separate ServiceWorkerScope() method to get the scope out of a WorkerPrivate.  It still uses the same underlying storage today.  This will make it easier to refactor the scope storage in the future.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e764099b6283d7b0d0cfbb22f0c7d54514d47173

Comment 2

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 8851099 [details] [diff] [review]
Expose WorkerPrivate::ServiceWorkerScope() instead of using WorkerName() all over the place. r=asuth

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

Regrettably, this limits clever comments referencing The Ting Tings' "That's Not My Name" going forward.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1e329519364b
Expose WorkerPrivate::ServiceWorkerScope() instead of using WorkerName() all over the place. r=asuth

Comment 4

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/1e329519364b