1483505 - Registering a service worker with a data: URL throws SecurityError instead of TypeError

Status: RESOLVED FIXED

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

Description

[Yaron Tausky [:ytausky]]

The spec requires a TypeError in this case: https://w3c.github.io/ServiceWorker/#start-register
This is currently not tested, as far as I can tell.

Comment 1

[Yaron Tausky [:ytausky]]

Attachment: Bug 1483505: Test ServiceWorkerContainer.register() with a data: URL

The proper response to a registration attempt with a data: URL is a TypeError,
yet we currently reject with a Security error. See spec.:
https://w3c.github.io/ServiceWorker/#start-register

Currently this test is expected to fail.

Comment 2

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

Comment on attachment 9000536 [details]
Bug 1483505: Test ServiceWorkerContainer.register() with a data: URL

Andrew Sutherland [:asuth] has approved the revision.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0e8f94baa607
Test ServiceWorkerContainer.register() with a data: URL r=asuth

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/12624 for changes under testing/web-platform/tests

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 6

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/0e8f94baa607

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged