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 TrustedType tests for iframe.srcdoc. #49996

Merged

Conversation

fred-wang
Copy link
Contributor

It seems we already have some tests for iframe.srcdoc but things are a bit messy so it's unclear whether we cover blocking, conversion check and sink names (#494) and null string edge case. Add a simple test doing that, similar to other block-string-assignment-to-* tests.

It seems we already have some tests for iframe.srcdoc but things are
a bit messy so it's unclear whether we cover blocking, conversion
check and sink names (#494) and null string edge case. Add a simple
test doing that, similar to other `block-string-assignment-to-*`
tests.
@fred-wang fred-wang requested a review from lukewarlow January 9, 2025 15:28
Copy link
Member

@lukewarlow lukewarlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm assuming it passes in WebKit and chrome (unless there's an actual reason why it fails)

@fred-wang
Copy link
Contributor Author

Lgtm assuming it passes in WebKit and chrome (unless there's an actual reason why it fails)

Yes, I verified the latest version in Chromium, Safari and Firefox.

@fred-wang fred-wang merged commit 15cf5d6 into master Jan 9, 2025
19 checks passed
@fred-wang fred-wang deleted the block-string-assignment-to-HTMLIFrameElement-srcdoc branch January 9, 2025 16:53
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