Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1823658] Fix the first expectation of pointerevent_after_target_removed.html?mouse to conform to the spec #44199

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

moz-wptsync-bot
Copy link
Collaborator

pointerup should not imply the pointer move and pointerover should be fired
only when the pointer is actually moved. Therefore, pointerover should be
fired after pointerup which is followed by another pointermove instead of
before pointerup.

Differential Revision: https://phabricator.services.mozilla.com/D199607

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1823658
gecko-commit: 6cfd68d8e76b8a3b82fb7224ca48e1ddff388e3b
gecko-reviewers: smaug

…mouse` to conform to the spec

`pointerup` should not imply the pointer move and `pointerover` should be fired
only when the pointer is actually moved. Therefore, `pointerover` should be
fired after `pointerup` which is followed by another `pointermove` instead of
before `pointerup`.

Differential Revision: https://phabricator.services.mozilla.com/D199607

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1823658
gecko-commit: 6cfd68d8e76b8a3b82fb7224ca48e1ddff388e3b
gecko-reviewers: smaug
Copy link
Collaborator

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

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 Firefox project.

@smaug----
Copy link
Contributor

The reasoning for the change is in web-platform-tests/interop#380

@jgraham
Copy link
Contributor

jgraham commented Jan 25, 2024

CC @foolip @ntim for visibility since this test is in Interop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants