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 missing headers file to worker eval TT test #50054

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

lukewarlow
Copy link
Member

Currently, this test will fail in all browsers because trusted types isn't enforced without the headers file.

Currently, this test will fail in all browsers because trusted types isn't enforced without the headers file.
@lukewarlow
Copy link
Member Author

I'm not convinced we neccessarily need this test given we have WorkerGlobalScope-eval already which can be extended with the FUnction constructor subtests, but for now this at least fixes it.

@fred-wang
Copy link
Contributor

I'm not convinced we neccessarily need this test given we have WorkerGlobalScope-eval already which can be extended with the FUnction constructor subtests, but for now this at least fixes it.

Right, probably we can make sure the compilationSink and Functions are tested in WorkerGlobalScope-eval.https.js

@fred-wang fred-wang merged commit 9b8e806 into master Jan 14, 2025
19 checks passed
@fred-wang fred-wang deleted the add-missing-headers-file branch January 14, 2025 07:05
sadym-chromium pushed a commit that referenced this pull request Jan 14, 2025
Currently, this test will fail in all browsers because trusted types isn't enforced without the headers file.
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