Skip to content

script: Reset click count when touchend doesn't synthesize mouse events - #48699

Merged
yezhizhen merged 1 commit into
servo:mainfrom
yezhizhen:fix-intermittent-again-
Oct 7, 2026
Merged

yezhizhen merged 1 commit into
servo:mainfrom
yezhizhen:fix-intermittent-again-

Conversation

@yezhizhen

@yezhizhen yezhizhen commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

/touch-events/mouseevents-after-touchend.tentative.html reveals an general issue that the side-effect of previous subtests remain: a touch sequence that doesn't synthesize compatibility mouse events doesn't reset click count.

That may turn two unrelated quick tap/two finger gesture into weird results.

Testing: /touch-events/mouseevents-after-touchend.tentative.html no longer intermittent.
Fixes: #46497

.
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
@yezhizhen
yezhizhen requested a review from gterzian as a code owner October 7, 2026 05:18
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 7, 2026
@yezhizhen
yezhizhen requested a review from TimvdLippe October 7, 2026 05:19
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 7, 2026
@yezhizhen
yezhizhen added this pull request to the merge queue Oct 7, 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 Oct 7, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 7, 2026
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Oct 7, 2026
@yezhizhen
yezhizhen added this pull request to the merge queue Oct 7, 2026
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Oct 7, 2026
Merged via the queue into servo:main with commit 877b798 Oct 7, 2026
43 checks passed
@yezhizhen
yezhizhen deleted the fix-intermittent-again- branch October 7, 2026 07:06
@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 Oct 7, 2026
hhhwsqhhh pushed a commit to hhhwsqhhh/servo that referenced this pull request Oct 12, 2026
…ts (servo#48699)

`/touch-events/mouseevents-after-touchend.tentative.html` reveals an
general issue that the side-effect of previous subtests remain: a touch
sequence that doesn't synthesize compatibility mouse events doesn't
reset click count.

That may turn two unrelated quick tap/two finger gesture into weird
results.

Testing: `/touch-events/mouseevents-after-touchend.tentative.html` no
longer intermittent.
Fixes:  servo#46497

Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
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.

Intermittent PASS in /touch-events/mouseevents-after-touchend.tentative.html

3 participants