Skip to content

script: Replace the InputEventId of existing_constellation_wheel_event when coalescing wheel events#43138

Merged
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:fix-blocking-wheel
Mar 10, 2026
Merged

script: Replace the InputEventId of existing_constellation_wheel_event when coalescing wheel events#43138
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:fix-blocking-wheel

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Mar 10, 2026

Basically, any testdriver tests with scroll duration
more than 16ms can be intermittent due to this.

Fixes: #43136
Fixes: #43110
Testing: This fixes the issue where newly added test
in #43126 randomly TIMEOUT with very high probability:
https://github.com/servo/servo/actions/runs/22906496902

.
Signed-off-by: Euclid Ye <[email protected]>
@yezhizhen yezhizhen requested a review from gterzian as a code owner March 10, 2026 14:18
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 10, 2026
@yezhizhen yezhizhen changed the title script: Replace the InputId of existing_constellation_wheel_event when coalescing events script: Replace the InputEventId of existing_constellation_wheel_event when coalescing events Mar 10, 2026
@yezhizhen yezhizhen changed the title script: Replace the InputEventId of existing_constellation_wheel_event when coalescing events script: Replace the InputEventId of existing_constellation_wheel_event when coalescing wheel events Mar 10, 2026
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.

Seems fine as a workaround though we will need to address this for real as well.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 10, 2026
@yezhizhen yezhizhen added this pull request to the merge queue Mar 10, 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 Mar 10, 2026
Merged via the queue into servo:main with commit 67e54cc Mar 10, 2026
38 checks passed
@yezhizhen yezhizhen deleted the fix-blocking-wheel branch March 10, 2026 15:16
@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 Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants