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 1655393] Add WPT for `InputEvent.getTargetRanges() #24866

Merged
merged 1 commit into from
Aug 4, 2020

Conversation

moz-wptsync-bot
Copy link
Collaborator

There are no automated tests for InputEvent.getTargetRanges() because it
is set only when beforeinput event, but it's defined as not dispatched by
document.execCommand.

However, we can synthesize beforeinput event with test driver.

On the other hand, the definition in Input Events spec is not clear.
Therefore, most of the tests won't be passed on any browsers for now.
There are some spec issues which I filed:

These new test must be useful when browser vendors discuss the issues.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1655393
gecko-commit: 12f21ad909371384939bb38d65cf1dbc797b5bf9
gecko-integration-branch: autoland
gecko-reviewers: smaug

There are no automated tests for `InputEvent.getTargetRanges()` because it
is set only when `beforeinput` event, but it's defined as not dispatched by
`document.execCommand`.

However, we can synthesize `beforeinput` event with test driver.

On the other hand, the definition in Input Events spec is not clear.
Therefore, most of the tests won't be passed on any browsers for now.
There are some spec issues which I filed:
* w3c/input-events#112
* w3c/input-events#113
* w3c/input-events#114

These new test must be useful when browser vendors discuss the issues.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1655393
gecko-commit: 12f21ad909371384939bb38d65cf1dbc797b5bf9
gecko-integration-branch: autoland
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.

@moz-wptsync-bot moz-wptsync-bot merged commit cc1a9c4 into master Aug 4, 2020
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1655393 branch August 4, 2020 09:56
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.

3 participants