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

Cherry-pick 2 ladybird PRs #25406

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

nico
Copy link
Contributor

@nico nico commented Nov 16, 2024

ADKaster and others added 7 commits November 16, 2024 09:40
This simply adds a ServiceWorkerContainer object to the WorkerNavigator.

(cherry picked from commit 2667726b74ae374be2907552795cf56639c4d57f)
(cherry picked from commit 4df280689b5c7f4df53cdcb1c61cb3fed99461c6)
This API is a relic from the time when it was important for objects to
have easy access to the Window, and to know it was the global object.

We now have more spec-related concepts like relevant_global_object and
current_global_object to pull the Window out of thin air.

(cherry picked from commit 02a56f648000517f191eeaea2abc68783c097486)
(cherry picked from commit ade4fc177920546c765c5802a57f6d0e9d7434f8)
(cherry picked from commit 0ece40c466dbd6c43fc9b0707d32b3d5d42fe751)
Looking at the spec it doesn't seem like there's a chance for a service
worker client to be an environment but not an environment settings
object. In the case that that changes in the implementation, we can
move it.

(cherry picked from commit 237bcbff360d5000ab45da2ca63f196064ab5210)
When a style sheet is removed, all font loaders created from that style
sheet should also be removed.

(cherry picked from commit 74588a0a16fd3a2cb10c50a58f47b9f5695bc2ff)
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 pr-needs-review PR needs review from a maintainer or community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants