Skip to content

Derive Clone on SourceSize and SourceSizeList - #440

Merged
jdm merged 1 commit into
servo:mainfrom
lumiscosity:srcset-derive-clone
Aug 11, 2026
Merged

Derive Clone on SourceSize and SourceSizeList#440
jdm merged 1 commit into
servo:mainfrom
lumiscosity:srcset-derive-clone

Conversation

@lumiscosity

Copy link
Copy Markdown
Member

All members of these structs already derive/implement Clone, so this is trivial. Required for cloning an HTMLLinkElement's source set when constructing LinkProcessingOptions.

All members of these classes already derive/implement Clone, so this is trivial. Required for cloning an HTMLLinkElement's source set when constructing LinkProcessingOptions.

Signed-off-by: lumiscosity <averyrudelphe@gmail.com>
@lumiscosity lumiscosity changed the title Derive clone on SourceSize and SourceSizeList Derive Clone on SourceSize and SourceSizeList Aug 10, 2026
@jdm

jdm commented Aug 10, 2026

Copy link
Copy Markdown
Member

We should submit this upstream as well, since it's a type that's shared with Gecko.

@jdm

jdm commented Aug 10, 2026

Copy link
Copy Markdown
Member

@jdm
jdm added this pull request to the merge queue Aug 11, 2026
Merged via the queue into servo:main with commit cc499a8 Aug 11, 2026
5 checks passed
AbdAlRahmanGad pushed a commit to AbdAlRahmanGad/servo that referenced this pull request Aug 14, 2026
Companion Stylo PR: servo/stylo#440
Implements source set operations on the `<link>` element.

Testing: 4 new passing in `preload`, attribute tests on idlharness pass.

---------

Signed-off-by: lumiscosity <averyrudelphe@gmail.com>
Co-authored-by: Tim van der Lippe <TimvdLippe@users.noreply.github.com>
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.

2 participants