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

LibWeb: Implement WorkerGlobalScope.importScripts() #24423

Merged

Conversation

tcl3
Copy link
Member

@tcl3 tcl3 commented May 23, 2024

This method allows workers to synchronously import one or more scripts. It is used by all *.worker.html WPT tests. Before this change all of these tests would time out, so this should yield a significant reduction in test timeouts.

tcl3 added 4 commits May 27, 2024 18:14
The `api_base_url()` and `policy_container()` getters now get their
respective objects from the global scope.
This method allows workers to synchronously import one or more scripts.
This fixes an issue where fetch requests to HTTP(s) addresses would
fail inside workers.
@tcl3 tcl3 force-pushed the libweb_workerglobalscope_importscripts branch from 27e9c57 to 4d9f72c Compare May 27, 2024 18:13
@tcl3 tcl3 marked this pull request as ready for review May 27, 2024 18:30
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label May 27, 2024
@awesomekling awesomekling merged commit d0f88d4 into SerenityOS:master May 29, 2024
15 checks passed
@github-actions github-actions bot removed the 👀 pr-needs-review PR needs review from a maintainer or community member label May 29, 2024
@tcl3 tcl3 deleted the libweb_workerglobalscope_importscripts branch May 29, 2024 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants