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
Remove StringContext attribute #498
base: main
Are you sure you want to change the base?
Conversation
@@ -1115,18 +1117,9 @@ To <dfn abstract-op export>get Trusted Types-compliant attribute value</dfn> on | |||
# Integrations # {#integrations} | |||
|
|||
<pre class="idl"> | |||
typedef [StringContext=TrustedHTML] DOMString HTMLString; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to remember to update the HTML, Dom Parser, and Sanitiser API specs to use union types instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As of w3c/DOM-Parsing#82 DOM Parsing spec will no longer contain any TT sinks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure it's worth another PR to the sanitiser api, when they're upstreaming they'll get the idea what's going on (especially given Anne has played a part in both specs)
8310136
to
0a79b5b
Compare
Swap to using a union type and updating the algorithms directly.
Swap to using a union type and updating the algorithms directly.
Need #457 merged first as this removes usages of StringContext
Corresponding HTML PR: whatwg/html#10286 (whatwg/html#10328)
Corresponding Sanitizer PR: TODO
Preview | Diff