Skip to content

script: Use unrooted iterator for stringify - #46090

Merged
TimvdLippe merged 3 commits into
servo:mainfrom
Narfinger:unroot-stringify
Jun 29, 2026
Merged

script: Use unrooted iterator for stringify#46090
TimvdLippe merged 3 commits into
servo:mainfrom
Narfinger:unroot-stringify

Conversation

@Narfinger

Copy link
Copy Markdown
Contributor

This moves the stringify method in dom/range and dom/selection to use the unrooted iterator.

Testing: This only changes the rooting behavior and, hence, compilation is the test.

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@Narfinger
Narfinger requested a review from gterzian as a code owner June 29, 2026 14:24
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jun 29, 2026
Comment thread components/script/dom/range/range.rs Outdated
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jun 29, 2026
Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jun 29, 2026
Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@TimvdLippe
TimvdLippe added this pull request to the merge queue Jun 29, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jun 29, 2026
Merged via the queue into servo:main with commit 26dce20 Jun 29, 2026
31 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jun 29, 2026
@Narfinger
Narfinger deleted the unroot-stringify branch June 30, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants