1825284 - "WebDriver:ElementClick" for <option> fires mouse events without expected parameters

Status: NEW

(Remote Protocol :: Marionette, defect, P3)

Description

[Oliver Medhurst [:canadahonk]]

The mouse events fired when doing "WebDriver:ElementClick" on an <option> element does not include the expected parameters of the mouse events fired (button, buttons, position, etc).

We should likely switch to using event.synthesizeMouseAtPoint instead of event.mousedown/etc in this case.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to CanadaHonk [:canadahonk] from comment #0)

We should likely switch to using event.synthesizeMouseAtPoint instead of event.mousedown/etc in this case.

We have a specific order of events that we fire when simulation the selection of an option. So if event.synthesizeMouseAtPoint is emitting the same events it might be fine, but otherwise we might have to set all the required fields in opts when calling the different mouse events directly. Note that you would also have to take bug 1550680 into account here.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:whimboo, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

CanadaHonk, would you be interested to help with this particular bug? It's blocking the other bug that you are working on and I'm not sure when I / we would get to this one. As usual I would assist where-ever I can. Thanks a lot.

Comment 4

[Oliver Medhurst [:canadahonk]]

Sure, just been busy as of late.

Comment 5

[Oliver Medhurst [:canadahonk]]

Attachment: Bug 1825284 - [marionette] Fire <option> click events with extra parameters

Previously mouse events fired when an <option> is clicked did not
include extra expected parameters (eg button).

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Oliver, would you have a chance to check my review comments which require an update of the patch? Thanks.

Comment 7

[Oliver Medhurst [:canadahonk]]

Sorry, I don't have the cycles to work on these patches at the moment.

Comment 8

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.