Skip to content

Avoid code injection in sandbox document-open mutation test#57764

Open
xu-cheng wants to merge 1 commit intoweb-platform-tests:masterfrom
xu-cheng:sandbox
Open

Avoid code injection in sandbox document-open mutation test#57764
xu-cheng wants to merge 1 commit intoweb-platform-tests:masterfrom
xu-cheng:sandbox

Conversation

@xu-cheng
Copy link
Contributor

Replace function-serialization injection with an inline sandbox check. Aligns the pattern with sandbox-document-open.html.

Replace function-serialization injection with an inline sandbox check.
Aligns the pattern with sandbox-document-open.html.
Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

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

Wouldn't it make more sense to make both tests use a common function?

Or perhaps I'm missing some part of the motivation here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants