Skip to content

script: Housekeeping WebIDL dictionaries of WebCrypto API#40652

Merged
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:tidy-up-aes
Nov 15, 2025
Merged

script: Housekeeping WebIDL dictionaries of WebCrypto API#40652
TimvdLippe merged 1 commit intoservo:mainfrom
kkoyung:tidy-up-aes

Conversation

@kkoyung
Copy link
Copy Markdown
Member

@kkoyung kkoyung commented Nov 15, 2025

Housekeeping of WebIDL dictionaries of WebCrypto API, including:

  • Add/Fix spec links in SubtleCrypto.webidl and CryptoKey.webidl.
  • Sort dictionaries in subtlecrypto.webidl based on the spec.
  • Sort the subtle structs in subtlecrypto.rs, based on the spec.
  • Reduce unneeded visibility of those subtle structs.

Testing: No behavioral change. Existing tests suffice.

Housekeeping of WebIDL dictionaries of WebCrypto API, including:

- Add/Fix spec links in `SubtleCrypto.webidl` and `CryptoKey.webidl`.
- Sort dictionaries in `subtlecrypto.webidl` based on the spec.
- Sort the `subtle` structs in `subtlecrypto.rs`, based on the spec.
- Reduce unneeded visibility of those `subtle` structs.

Testing: No behavioral change. Existing tests suffice.

Signed-off-by: Kingsley Yung <[email protected]>
@kkoyung kkoyung requested a review from gterzian as a code owner November 15, 2025 09:59
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 15, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 15, 2025
@TimvdLippe TimvdLippe added this pull request to the merge queue Nov 15, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 15, 2025
Merged via the queue into servo:main with commit 1ec01cc Nov 15, 2025
35 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 15, 2025
@kkoyung kkoyung deleted the tidy-up-aes branch November 15, 2025 12:56
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.

3 participants