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

Referenced SVG elements in an external resource document match :target #46545

Merged
merged 1 commit into from
May 30, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented May 30, 2024

Track which elements are referenced (by <use>) in an external resource
document by funneling the lookup through a special function on
SVGResourceDocumentContent that creates SVGResourceTarget objects for
each (valid) reference and lets the targeted element weakly reference
the object. While matching :target, the selector checker will check if
the element has a reference to an SVGResourceTarget object and let
:target match if it does.

Bug: 342705674
Change-Id: I9b88e4338dccc0e01c45afb0df3e1c412b2466a9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5574989
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/main@{#1307926}

Track which elements are referenced (by <use>) in an external resource
document by funneling the lookup through a special function on
SVGResourceDocumentContent that creates SVGResourceTarget objects for
each (valid) reference and lets the targeted element weakly reference
the object. While matching :target, the selector checker will check if
the element has a reference to an SVGResourceTarget object and let
:target match if it does.

Bug: 342705674
Change-Id: I9b88e4338dccc0e01c45afb0df3e1c412b2466a9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5574989
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/main@{#1307926}
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 c8838c8 into master May 30, 2024
15 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5574989 branch May 30, 2024 11:22
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.

None yet

2 participants