Skip to content

wpt: Fix syntax error in pointerevent_touch-propagates-when-target-is-video_touch.html#42405

Merged
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:fix-test
Feb 6, 2026
Merged

wpt: Fix syntax error in pointerevent_touch-propagates-when-target-is-video_touch.html#42405
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:fix-test

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Feb 6, 2026

The syntax is wrong. We cannot pass an Element as duration. Only Firefox was passing this,
which is a bug of geckodriver:

Testing: Updated expectation of existing tests.
This blocks #42398, #42387.

Signed-off-by: Euclid Ye <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 6, 2026
@yezhizhen yezhizhen requested a review from xiaochengh February 6, 2026 13:31
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

2 similar comments
@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@servo-wpt-sync
Copy link
Copy Markdown
Collaborator

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

@yezhizhen yezhizhen added this pull request to the merge queue Feb 6, 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 6, 2026
Merged via the queue into servo:main with commit 5a5d71d Feb 6, 2026
35 checks passed
@yezhizhen yezhizhen deleted the fix-test branch February 6, 2026 15:00
@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 6, 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

Development

Successfully merging this pull request may close these issues.

4 participants