Skip to content

Invalidate iterator over elements of a XPathResult when the document changes#39411

Merged
simonwuelker merged 3 commits intoservo:mainfrom
simonwuelker:xpath-result-iterator
Sep 24, 2025
Merged

Invalidate iterator over elements of a XPathResult when the document changes#39411
simonwuelker merged 3 commits intoservo:mainfrom
simonwuelker:xpath-result-iterator

Conversation

@simonwuelker
Copy link
Copy Markdown
Contributor

Also includes a fix to not throw a type error in XPathResult.invalidIteratorState.

Testing: Includes a new web platform test
Part of #34527

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

🤖 Opened new upstream WPT pull request (web-platform-tests/wpt#54967) with upstreamable changes.

@simonwuelker simonwuelker mentioned this pull request Sep 20, 2025
5 tasks
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54967).

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54967).

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Sep 24, 2025
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Sep 24, 2025
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54967).

@simonwuelker simonwuelker added this pull request to the merge queue Sep 24, 2025
@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 Sep 24, 2025
Merged via the queue into servo:main with commit 2ccaf86 Sep 24, 2025
28 checks passed
@simonwuelker simonwuelker deleted the xpath-result-iterator branch September 24, 2025 17:27
@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 Sep 24, 2025
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