Skip to content

script: Move worker DOM interfaces to script/dom/workers/#39718

Merged
sagudev merged 2 commits intoservo:mainfrom
Gae24:add-workers-module
Oct 8, 2025
Merged

script: Move worker DOM interfaces to script/dom/workers/#39718
sagudev merged 2 commits intoservo:mainfrom
Gae24:add-workers-module

Conversation

@Gae24
Copy link
Copy Markdown
Contributor

@Gae24 Gae24 commented Oct 8, 2025

Moves code related to the Web Worker spec to a new mod.

Testing: A successful build is enough
Part of #38901

@Gae24 Gae24 requested a review from gterzian as a code owner October 8, 2025 17:06
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 8, 2025
@Gae24
Copy link
Copy Markdown
Contributor Author

Gae24 commented Oct 8, 2025

worklet should be moved, but what about paintworkletglobalscope.rs?

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 8, 2025
@sagudev
Copy link
Copy Markdown
Member

sagudev commented Oct 8, 2025

worklet should be moved, but what about paintworkletglobalscope.rs?

I think not, because paintworklet comes from different spec: https://drafts.css-houdini.org/css-paint-api/#paintworkletglobalscope

@sagudev sagudev added this pull request to the merge queue Oct 8, 2025
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. S-awaiting-review There is new code that needs to be reviewed. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Oct 8, 2025
Signed-off-by: Gae24 <[email protected]>
@Gae24 Gae24 force-pushed the add-workers-module branch from 2ae03a8 to d2df09d Compare October 8, 2025 18:30
Merged via the queue into servo:main with commit b270387 Oct 8, 2025
2 checks passed
@Gae24 Gae24 deleted the add-workers-module branch October 21, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants