Skip to content

webdriver: Implement pointerCancel for touch chain#41937

Merged
yezhizhen merged 6 commits intoservo:mainfrom
yezhizhen:pointercancel-touch
Jan 17, 2026
Merged

webdriver: Implement pointerCancel for touch chain#41937
yezhizhen merged 6 commits intoservo:mainfrom
yezhizhen:pointercancel-touch

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Jan 15, 2026

Testing: Adding a new function pointer_cancel in test infrastructure and two touch chain subtests for pointerCancel.

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from c2b59e7 to 3febd21 Compare January 16, 2026 00:41
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-awaiting-review There is new code that needs to be reviewed. labels Jan 16, 2026
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 81aaf96 to 3c5d101 Compare January 16, 2026 07:37
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 3c5d101 to 23ca020 Compare January 16, 2026 10:11
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 23ca020 to 9e70cb4 Compare January 16, 2026 11:55
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@jdm jdm left a comment

Choose a reason for hiding this comment

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

Nice and clear!

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 16, 2026
@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 9e70cb4 to 2895485 Compare January 16, 2026 13:31
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 16, 2026
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 2895485 to 959ecb4 Compare January 16, 2026 13:43
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

✍ Updated existing upstream WPT pull request (web-platform-tests/wpt#57196) title and body.

@yezhizhen yezhizhen added this pull request to the merge queue Jan 17, 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 Jan 17, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 17, 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 Jan 17, 2026
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
@yezhizhen yezhizhen force-pushed the pointercancel-touch branch from 959ecb4 to 0af40bd Compare January 17, 2026 01:29
@servo-highfive servo-highfive removed the S-tests-failed The changes caused existing tests to fail. label Jan 17, 2026
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

1 similar comment
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

✍ Updated existing upstream WPT pull request (web-platform-tests/wpt#57196) title and body.

@yezhizhen yezhizhen added this pull request to the merge queue Jan 17, 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 Jan 17, 2026
Merged via the queue into servo:main with commit e991d16 Jan 17, 2026
30 checks passed
@yezhizhen yezhizhen deleted the pointercancel-touch branch January 17, 2026 03:04
@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 Jan 17, 2026
Taym95 pushed a commit to Taym95/servo that referenced this pull request Feb 20, 2026
…chend`/`touchcancel` (servo#42654)

[Spec](https://w3c.github.io/touch-events/#dfn-touchend:~:text=screen%2E-,The,element,-%2E)
for touchmove/touchend/touchcancel:
> The target of this event must be the same Element on which the [touch
point](https://w3c.github.io/touch-events/#dfn-touch-point) started when
it was first placed on the surface, even if the [touch
point](https://w3c.github.io/touch-events/#dfn-touch-point) has since
moved outside the interactive area of the target element.

Also, previously `touchend` can be fired after `touchcancel`, which was
wrong and fixed in this PR.

Testing: Fully passes `/touch-events/multi-touch-interfaces.html` (511
subtests) and new passes in
`pointerevents\compat\pointerevent_touch_target_after_pointerdown_target_removed.tentative.html`.

We also take the chance to update `pointercancel` related tests
introduced in servo#41937: `touchcancel`
should not be followed by `touchend` according to
[spec](https://w3c.github.io/touch-events/#dfn-touchcancel).

---------

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

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.

5 participants