Skip to content

script: Implement forwarddelete command#61388

Merged
jdm merged 1 commit into
web-platform-tests:masterfrom
servo:servo_export_46608
Jul 20, 2026
Merged

script: Implement forwarddelete command#61388
jdm merged 1 commit into
web-platform-tests:masterfrom
servo:servo_export_46608

Conversation

@servo-wpt-sync

Copy link
Copy Markdown
Collaborator

This implements (most of) the forwarddelete command. Singular TODO about forwarddelete'ing stuff with combining diacritics and such.

Testing: They pass now. Better yet, none of them crash anymore. There is one new timeout though.

I've also taken the liberty to basically rewrite forwarddelete-after-editable-slot-element-outside-body.html, since we failed it and the test case was a complete mess. Apparently the Firefox people got that one from a fuzzer some years ago and decided to just dump it into WPT without any cleanup: https://bugzilla.mozilla.org/show_bug.cgi?id=1706145
Reviewed in servo/servo#46608

@wpt-pr-bot wpt-pr-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Servo project.

@servo-wpt-sync
servo-wpt-sync force-pushed the servo_export_46608 branch 2 times, most recently from 406f4b9 to 3fe3018 Compare July 19, 2026 17:10
Signed-off-by: Psychpsyo (Cameron) <60073468+Psychpsyo@users.noreply.github.com>
@servo-wpt-sync servo-wpt-sync added stale-servo-export PRs that were supposed to merge but were not able to do so. and removed do not merge yet labels Jul 20, 2026
@servo-wpt-sync

Copy link
Copy Markdown
Collaborator Author

⛔ The downstream PR has merged (servo/servo#46608), but these changes could not be merged properly. Please address any CI issues and try to merge manually.

@TimvdLippe

Copy link
Copy Markdown
Contributor

Seems like CI hasn't finished yet. @jdm can you merge this one once it's all green?

@jdm
jdm merged commit beed4bd into web-platform-tests:master Jul 20, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editing servo-export stale-servo-export PRs that were supposed to merge but were not able to do so.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants