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.

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

Rename serializable to shadowrootserializable #45142

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Mar 15, 2024

See the conversation here:
whatwg/html#10139

But the TL;DR is that the \<template> attribute corresponding
to the declarative shadow root serializable attribute should
be named in a parallel way to other DSD-related attributes,
meaning it should start with shadowroot*.

Bug: 41490936
Change-Id: I40ea496c22147bac7dcdfb74cd7bf4a3abae8be9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5375416
Commit-Queue: Di Zhang <[email protected]>
Reviewed-by: Di Zhang <[email protected]>
Auto-Submit: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1274374}

See the conversation here:
  whatwg/html#10139

But the TL;DR is that the `<template>` attribute corresponding
to the declarative shadow root `serializable` attribute should
be named in a parallel way to other DSD-related attributes,
meaning it should start with `shadowroot*`.

Bug: 41490936
Change-Id: I40ea496c22147bac7dcdfb74cd7bf4a3abae8be9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5375416
Commit-Queue: Di Zhang <[email protected]>
Reviewed-by: Di Zhang <[email protected]>
Auto-Submit: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1274374}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit f6626d7 into master Mar 18, 2024
17 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5375416 branch March 18, 2024 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants