1433545 - Add scriptURL string to ServiceWorkerDescriptor

Status: RESOLVED FIXED

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

Description

[Ben Kelly [:bkelly, not reviewing]]

I forgot to add the scriptURL field in bug 1412858.  We don't need it yet, but we will for bug 1429200.

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Attachment: P1 Add scriptURL to ServiceWorkerDescriptor. r=asuth

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Attachment: P2 Remove mScriptSpec from ServiceWorkerInfo and use mDescriptor.ScriptURL() instead. r=asuth

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

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8946418 [details] [diff] [review]
P1 Add scriptURL to ServiceWorkerDescriptor. r=asuth

Add the script URL to the ServiceWorkerDescriptor.

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8946419 [details] [diff] [review]
P2 Remove mScriptSpec from ServiceWorkerInfo and use mDescriptor.ScriptURL() instead. r=asuth

This removes the ServiceWorkerInfo::mScriptSpec member in favor of using mDescriptor.ScriptURL() instead.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9be09d7a6297
P1 Add scriptURL to ServiceWorkerDescriptor. r=asuth
https://hg.mozilla.org/integration/mozilla-inbound/rev/a1b1e70fd362
P2 Remove mScriptSpec from ServiceWorkerInfo and use mDescriptor.ScriptURL() instead. r=asuth

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/9be09d7a6297
https://hg.mozilla.org/mozilla-central/rev/a1b1e70fd362