[wpt-sync] Sync PR 15512 - [ChromeDriver] Set key modifiers with mouse actions
Categories
(Testing :: web-platform-tests, enhancement, P4)
Tracking
(firefox68 fixed)
Tracking | Status | |
---|---|---|
firefox68 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 15512 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/15512
Details from upstream follow.
Lan Wei <lanwei@chromium.org> wrote:
[ChromeDriver] Set key modifiers with mouse actions
When we have keyboard events and mouse events send together, we should
keep the keys that we are pressing now and set them to the mouse events
that are in the same action sequence.Bug: chromedriver:1897
Change-Id: I700d7a87d3d0a1b45c1588318ab21314aa284417
Reviewed-on: https://chromium-review.googlesource.com/c/1482009
Reviewed-by: John Chen \<johnchen@chromium.org>
Commit-Queue: Lan Wei \<lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634700}--
Follow-up commit also included in this PR:
[wpt] Fix actionsWithKeyPressed.html.ini
actionsWithKeyPressed.html test does not work on both Firefox and
Safari, so I change actionsWithKeyPressed.html.ini to skip both
browsers.Bug: chromedriver:1897
Change-Id: I86fb1507cf86f2fe209414615b8ea73834456202
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529624
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#642112}
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=65fb41e04e2961f7f8f4ede35ec4e01bb9b5c655
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/d2c21624bc9a [wpt PR 15512] - [ChromeDriver] Set key modifiers with mouse actions, a=testonly
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/a4ca06c20ce4 [wpt PR 15512] - [ChromeDriver] Set key modifiers with mouse actions, a=testonly
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d2c21624bc9a
https://hg.mozilla.org/mozilla-central/rev/a4ca06c20ce4
Description
•