Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for ServiceWorkerContainer.register() from a Worker context. #50018

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

fred-wang
Copy link
Contributor

See https://w3c.github.io/ServiceWorker/#serviceworkercontainer-interface

Calling ServiceWorkerContainer.register() from a Window context is already covered by worker-constructor.html.

This commit covers the case when it's called from a Worker context, more specifically particular dedicated, shared and service workers.

This passes in Firefox with the patch from
https://bugzilla.mozilla.org/show_bug.cgi?id=1940044 and https://bugzilla.mozilla.org/show_bug.cgi?id=1901492.

See https://w3c.github.io/ServiceWorker/#serviceworkercontainer-interface

Calling ServiceWorkerContainer.register() from a Window context is
already covered by worker-constructor.html.

This commit covers the case when it's called from a Worker context,
more specifically particular dedicated, shared and service workers.

This passes in Firefox with the patch from
https://bugzilla.mozilla.org/show_bug.cgi?id=1940044 and
https://bugzilla.mozilla.org/show_bug.cgi?id=1901492.
@fred-wang fred-wang merged commit f371b41 into master Jan 10, 2025
16 checks passed
@fred-wang fred-wang deleted the serviceworkercontainer-register branch January 10, 2025 11:52
sadym-chromium pushed a commit that referenced this pull request Jan 14, 2025
…#50018)

See https://w3c.github.io/ServiceWorker/#serviceworkercontainer-interface

Calling ServiceWorkerContainer.register() from a Window context is
already covered by worker-constructor.html.

This commit covers the case when it's called from a Worker context,
more specifically particular dedicated, shared and service workers.

This passes in Firefox with the patch from
https://bugzilla.mozilla.org/show_bug.cgi?id=1940044 and
https://bugzilla.mozilla.org/show_bug.cgi?id=1901492.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants