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

Revert "Add a runtime enabled feature for removal of data: URL in SVGUseElement" #37854

Closed
wants to merge 1 commit into from

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

This reverts commit 23fdc0eb00c8597c2c74faee91b4a6a4e2296f3c.

Reason for revert: Not landing immediately, I want to do a manual try on GPU bots to confirm if this is the cause of an SVG test failure. See crbug.com/1406342

Original change's description:

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

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 <[email protected]>
Commit-Queue: Jun Kokatsu <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1090945}

Change-Id: I09bed766cad4afcf99e526af82d0736c39190de9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

Reviewed-on: https://chromium-review.googlesource.com/4153050
WPT-Export-Revision: 39a14797ce339a430606af305019d43a3b5978ca

…UseElement"

This reverts commit 23fdc0eb00c8597c2c74faee91b4a6a4e2296f3c.

Reason for revert: Not landing immediately, I want to do a manual try on GPU bots to confirm if this is the cause of an SVG test failure. See crbug.com/1406342

Original change's description:
> Add a runtime enabled feature for removal of data: URL in SVGUseElement
>
> 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 <[email protected]>
> Commit-Queue: Jun Kokatsu <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#1090945}

Change-Id: I09bed766cad4afcf99e526af82d0736c39190de9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
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
Copy link
Collaborator Author

Close this PR because the Chromium CL has been abandoned.

@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-4153050 branch January 25, 2023 14:12
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.

2 participants