1613571 - Fix some more issues with service worker exceptions

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, task)

Description

[Boris Zbarsky [:bzbarsky]]

ServiceWorkerScopeAndScriptAreValid doesn't quite do what the spec says in various ways.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Bug 1613571. Align service worker invalid-script-url and invalid-scope-url checks better with the spec.

Behavior changes:

1. We now throw a TypeError, not a SecurityError, when the script URI scheme is
   not 'http' or 'https', like the spec says to.

2. We now throw when the scope URI scheme is not 'http' or 'https', like the
   spec says to.

3. We now throw a proper TypeError, not a DOMException, at the end of
   CheckForSlashEscapedCharsInPath.

4. More informative error messages.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e2543a95ade0
Align service worker invalid-script-url and invalid-scope-url checks better with the spec. r=dom-workers-and-storage-reviewers,perry

Comment 3

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

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

Comment 4

[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 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/e2543a95ade0

Comment 6

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

Upstream PR merged by moz-wptsync-bot