Skip to content

Implement inner slot for cryptographic nonce#36965

Merged
jdm merged 1 commit intoservo:mainfrom
TimvdLippe:pr-timvdlippe-nonce-internal-slot
May 14, 2025
Merged

Implement inner slot for cryptographic nonce#36965
jdm merged 1 commit intoservo:mainfrom
TimvdLippe:pr-timvdlippe-nonce-internal-slot

Conversation

@TimvdLippe
Copy link
Copy Markdown
Contributor

@TimvdLippe TimvdLippe commented May 11, 2025

Also update the html/dom/reflection-metadata.html test
to handle the case where nonce does not reflect back
to the attribute after an IDL change.

Part of #4577
Fixes web-platform-tests/wpt#43286

@TimvdLippe TimvdLippe requested a review from gterzian as a code owner May 11, 2025 20:58
[Writing 'nonce' IDL attribute.]
expected: FAIL

[createElement.nonce.]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am a bit confused why this one isn't passing, but it's been a long evening. So need to investigate later what's going on.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It has to do with document.currentScript pointing to the wrong value. I think this is not a problem with handling nonces, but a problem in the JS engine. Therefore, let's leave this test failing for now until the JS engine updates are there.

@TimvdLippe
Copy link
Copy Markdown
Contributor Author

@TimvdLippe
Copy link
Copy Markdown
Contributor Author

Try run has test failures, but they are expected atm: web-platform-tests/wpt#43286 Will look into updating these tests to reflect (no pun intended) the new behavior.

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

🤖 Opened new upstream WPT pull request (web-platform-tests/wpt#52506) with upstreamable changes.

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

✍ Updated existing upstream WPT pull request (web-platform-tests/wpt#52506) title and body.

@TimvdLippe TimvdLippe requested a review from jdm May 13, 2025 19:46
@TimvdLippe TimvdLippe force-pushed the pr-timvdlippe-nonce-internal-slot branch from cc9cc60 to d611ea8 Compare May 13, 2025 20:18
@TimvdLippe
Copy link
Copy Markdown
Contributor Author

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#52506).

@TimvdLippe TimvdLippe force-pushed the pr-timvdlippe-nonce-internal-slot branch from d611ea8 to 23e568e Compare May 14, 2025 07:37
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#52506).

Also update the `html/dom/reflection-metadata.html` test
to handle the case where `nonce` does not reflect back
to the attribute after an IDL change.

Part of servo#4577
Fixes web-platform-tests/wpt#43286

Signed-off-by: Tim van der Lippe <[email protected]>
@TimvdLippe TimvdLippe force-pushed the pr-timvdlippe-nonce-internal-slot branch from 23e568e to 17d51d3 Compare May 14, 2025 09:49
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#52506).

@jdm jdm enabled auto-merge May 14, 2025 10:16
@jdm jdm added this pull request to the merge queue May 14, 2025
Merged via the queue into servo:main with commit a24fce3 May 14, 2025
21 checks passed
@TimvdLippe TimvdLippe deleted the pr-timvdlippe-nonce-internal-slot branch May 14, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

html/dom/reflection-metadata.html needs adjusting

3 participants