Closed Bug 1498383 Opened 6 years ago Closed 6 years ago

[wdspec] clear.py relies on old Gecko's <button> hit testing

Categories

(Testing :: geckodriver, enhancement)

enhancement
Not set
normal

Tracking

(firefox66 fixed)

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

Details

Attachments

(1 file)

test_button_with_subtree seems to rely on the old behavior
Attached patch clear.py_button_export.diff (deleted) — Splinter Review
Attachment #9016587 - Flags: review?(hskupin)
Attachment #9016587 - Flags: review?(hskupin) → review?(ato)
Comment on attachment 9016587 [details] [diff] [review] clear.py_button_export.diff Review of attachment 9016587 [details] [diff] [review]: ----------------------------------------------------------------- Thanks Olli!
Attachment #9016587 - Flags: review?(ato) → review+
Interesting that no other browser vendor, and I assume have already implemented that behavior, has complaint about this test yet.
Component: web-platform-tests → geckodriver
Summary: clear.py relies on old Gecko's <button> hit testing → [wdspec] clear.py relies on old Gecko's <button> hit testing
Hmm, this test gets run on all the browsers? The test is basically data:text/html,<button><input value=foobar></button> and one is supposed to be able to use that input element. And can in current Chrome, and in FF with the patch for bug 1089326
If it’s of any help, it looks like other vendors run test_button_with_subtree but that the test in its current state is failing for them: https://wpt.fyi/results/webdriver/tests/element_clear/clear.py?label=experimental I suppose we will get the real answer to whether or not they implement the correct behaviour when this patch lands, and wpt.fyi does another overnight test run.
Yes, that is indeed correct. Lets see how this changes when the patch landed.
Pushed by opettay@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/7029e8b4d7d0 clear.py relies on old Gecko's <button> hit testing, r=ato
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/14477 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: