Skip to content

Remove accidentally re-added logic to WebDriverSession::input_cancel_list#37081

Merged
mrobinson merged 1 commit intoservo:mainfrom
yezhizhen:fix-regression
May 23, 2025
Merged

Remove accidentally re-added logic to WebDriverSession::input_cancel_list#37081
mrobinson merged 1 commit intoservo:mainfrom
yezhizhen:fix-regression

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented May 22, 2025

Re-remove deleted logic from WebDriverSession::input_cancel_list logic in #37010 which was re-adedd during merge resolution in #36932. Otherwise, duplicate items would be registered in Release Actions.

@yezhizhen yezhizhen changed the title Bring back logic in #37010 which was regressed by #36932 Bring back WebDriverSession::input_cancel_list logic in #37010 which was regressed by #36932 May 22, 2025
@mrobinson
Copy link
Copy Markdown
Member

I'm happy to review this, but I'm a bit confused, because the description of the change is "bring back logic" but this just removes code. Do you mind clarifying?

@yezhizhen
Copy link
Copy Markdown
Member Author

yezhizhen commented May 23, 2025

Yes, the removing here is part of the logic of #37010.
It is wrongly added back in #36932 when resolving conflicts, I believe.

@mrobinson mrobinson changed the title Bring back WebDriverSession::input_cancel_list logic in #37010 which was regressed by #36932 Remove accidentally re-added logic to WebDriverSession::input_cancel_list May 23, 2025
Copy link
Copy Markdown
Member

@mrobinson mrobinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this fix!

@mrobinson mrobinson added this pull request to the merge queue May 23, 2025
Merged via the queue into servo:main with commit 6d04ef2 May 23, 2025
28 checks passed
@yezhizhen yezhizhen deleted the fix-regression branch August 8, 2025 07:16
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