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.

Sign up for GitHub

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

Jump to bottom

Remove callerRealm param from EnsureCSPDoesNotBlockStringCompilation #541

Merged
merged 1 commit into from
Apr 22, 2022
Merged

Remove callerRealm param from EnsureCSPDoesNotBlockStringCompilation #541

merged 1 commit into from
Apr 22, 2022

Conversation

antosart
Copy link
Member

This PR cleans up the EnsureCSPDoesNotBlockStringCompilation algorithm removing the callerRealm parameter, which is not used.

See also #540 and #438.

@antosart antosart mentioned this pull request Feb 23, 2022
Merged
3 tasks
@antosart
Copy link
Member Author

Companion PRs on html whatwg/html#7653 and on ECMAScript tc39/ecma262#2670.

@antosart antosart requested a review from annevk February 23, 2022 10:47
annevk pushed a commit to whatwg/html that referenced this pull request Apr 22, 2022
@antosart
Remove callerRealm from HostEnsureCanCompileStrings
fecc699 
See tc39/ecma262#2670 and w3c/webappsec-csp#541 for complementary PRs.
@antosart antosart merged commit 4430aa5 into w3c:main Apr 22, 2022
@antosart antosart deleted the cleanup-caller-realm branch April 22, 2022 09:10
github-actions bot added a commit that referenced this pull request Apr 22, 2022
@antosart @github-actions
Remove callerRealm param from EnsureCSPDoesNotBlockStringCompilation (#…
8105039 
…541)

SHA: 4430aa5
Reason: push, by @antosart

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
mfreed7 pushed a commit to mfreed7/html that referenced this pull request Jun 3, 2022
@antosart @mfreed7
Remove callerRealm from HostEnsureCanCompileStrings
e7907f4 
See tc39/ecma262#2670 and w3c/webappsec-csp#541 for complementary PRs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@annevk annevk annevk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@antosart @annevk