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

Add a runtime enabled feature for removal of data: URL in SVGUseElement #37511

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Dec 14, 2022

This CL adds a runtime enabled feature and WPT for removal of data: URLs
in in SVGUseElement.
See: w3c/svgwg#901

But: 1300195
Change-Id: Ieab699fc167b254c65b92ed811756404dcc7c575
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4104249
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Jun Kokatsu <jkokatsu@google.com>
Cr-Commit-Position: refs/heads/main@{#1090945}

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 force-pushed the chromium-export-cl-4104249 branch 5 times, most recently from 81591e9 to 77a3e7d Compare December 17, 2022 03:01
@chromium-wpt-export-bot chromium-wpt-export-bot changed the title Add Web Platform tests for SVGUseElement Add a runtime enabled feature for removal of data: URL in SVGUseElement Dec 19, 2022
This CL adds a runtime enabled feature and WPT for removal of data: URLs
in in SVGUseElement.
See: w3c/svgwg#901

But: 1300195
Change-Id: Ieab699fc167b254c65b92ed811756404dcc7c575
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4104249
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Jun Kokatsu <jkokatsu@google.com>
Cr-Commit-Position: refs/heads/main@{#1090945}
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.

4 participants