Open Bug 1942016 Opened 14 days ago Updated 7 hours ago

test_driver.click and test_driver.Actions can click on elements that are unclickable from using the UI (a.k.a the mouse)

Categories

(Testing :: web-platform-tests, defect)

defect

Tracking

(Not tracked)

People

(Reporter: farre, Unassigned)

References

(Depends on 1 open bug)

Details

In Bug 1935609 we found issues with <object> elements inside an <a> element that embedded a cross-origin image becoming unclickable. This was partially fixed, but when trying to extend testing/web-platform/tests/html/semantics/embedded-content/the-object-element/object-image-click.sub.html to test this issues with test_driver was found.

Adding cases for exactly what was described above, something like:

<a href="#cross-origin-implicit"><object id="cross-origin-implicit" data="http://{{hosts[alt][www]}}:{{ports[http][0]}}/images/green.png"></object></a>

would happily navigate to that hash, even though clicking on the <object> element manually fails to do so. And this regardless of whether test_driver.click was used or if the click was composed manually using test_driver.Actions. It should be noted that there is a sliver of the anchor sticking out at the bottom of the element, and this is clickable.

See Also: → 1935609
Summary: test_driver.click and test_driver.Actions can click on elements that are unclickable in the UI → test_driver.click and test_driver.Actions can click on elements that are unclickable from using the UI (a.k.a the mouse)

Do we use apz for click events? I think that might be the difference

Flags: needinfo?(hskupin)

No, we do not use APZ yet but directly emit events via EventUtils on the currently selected browsing context. Currently I'm working towards switching our code to widget events that are getting emitted via the parent process. This work is happening on bug 1773393. Note that it will still take a bit until we have it done and enabled by default.

Depends on: parent-actions
Flags: needinfo?(hskupin)

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

For more information, please visit BugBot documentation.

Flags: needinfo?(james)

I think this is effectively a duplicate of bug 1773393, but I suppose we can leave it open to cover whatever's required to move wpt to the new configuration once it's ready.

Severity: -- → S3
Flags: needinfo?(james)
You need to log in before you can comment on or make changes to this bug.