Skip to content

webdriver: Remove orphan Handler::num_pending_actions#42297

Merged
yezhizhen merged 2 commits intoservo:mainfrom
yezhizhen:remove-orphan
Feb 3, 2026
Merged

webdriver: Remove orphan Handler::num_pending_actions#42297
yezhizhen merged 2 commits intoservo:mainfrom
yezhizhen:remove-orphan

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Feb 3, 2026

Handler::num_pending_actions has been a poor orphan since who-knows-when.
This also removes the only dependency on Cell.

Testing: This removes unused field. Covered by existing tests.

@yezhizhen yezhizhen requested a review from xiaochengh February 3, 2026 04:15
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 3, 2026
@servo-highfive servo-highfive added S-needs-rebase There are merge conflict errors. and removed S-awaiting-review There is new code that needs to be reviewed. labels Feb 3, 2026
Signed-off-by: Euclid Ye <[email protected]>
Signed-off-by: Euclid Ye <[email protected]>
@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-needs-rebase There are merge conflict errors. labels Feb 3, 2026
@yezhizhen yezhizhen enabled auto-merge February 3, 2026 05:33
@yezhizhen yezhizhen added this pull request to the merge queue Feb 3, 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 Feb 3, 2026
Merged via the queue into servo:main with commit d2b4b03 Feb 3, 2026
32 checks passed
@yezhizhen yezhizhen deleted the remove-orphan branch February 3, 2026 06:33
@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 Feb 3, 2026
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.

3 participants