Remove Webdriver command "ElementTap"

RESOLVED FIXED in Firefox 64

Status

enhancement
P1
normal
RESOLVED FIXED
a year ago
8 months ago

People

(Reporter: whimboo, Assigned: ato, Mentored)

Tracking

(Blocks 1 bug)

unspecified
mozilla64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [lang=rust])

Attachments

(1 attachment, 1 obsolete attachment)

There is a "ElementTap" command present in both geckodriver and webdriver. But there is nothing like that listed in the WebDriver spec. In Marionette it calls `singleTap`, which is part of the old action API.

Not sure when or why this has been added in the past. Maybe it was part of the spec at an earlier stage?

Andreas, am I correct that this can be removed?
Flags: needinfo?(ato)
Assignee

Comment 1

a year ago
This can probably be removed.
Flags: needinfo?(ato)
Great. So all references to this command can then be removed from any of the files under `testing/geckodriver` and `testing/webdriver`.
Mentor: hskupin
Whiteboard: [lang=rust]

Comment 3

a year ago
Here is a patch removing ElementTap.
Attachment #8967982 - Flags: review?(hskupin)
Comment on attachment 8967982 [details] [diff] [review]
0001-1-remove-ElementTap.patch

Review of attachment 8967982 [details] [diff] [review]:
-----------------------------------------------------------------

The patch looks fine and you should have covered each instance of ElementTap. But the commit message is incorrect because it references Marionette, and not geckodriver.

Also maybe you could push your updated patch to mozreview? That would be fantastic because then I could easily create a try build to verify the changes. Thanks.
Attachment #8967982 - Flags: review?(hskupin) → review-
Assignee: nobody → keremkat
Status: NEW → ASSIGNED
Assignee

Updated

a year ago
Priority: -- → P3
Assignee

Comment 5

a year ago
Do you have time and ability to fix up the patch?
Flags: needinfo?(keremkat)

Comment 6

a year ago
Unfortunately, not currently.
Flags: needinfo?(keremkat)
Andreas, given that it was only the commit message which is wrong, maybe you can update and trigger a try build?
Assignee

Updated

8 months ago
Assignee: keremkat → ato
Priority: P3 → P1
Assignee

Updated

8 months ago
Assignee

Updated

8 months ago
Attachment #8967982 - Attachment is obsolete: true
Assignee

Comment 8

8 months ago
Removes the non-standard /session/{session id}/element/{element
id}/tap endpoint command from geckodriver.
Comment on attachment 9012046 [details]
bug 1453292: webdriver, geckodriver: remove elementTap command

Henrik Skupin (:whimboo) has approved the revision.
Attachment #9012046 - Flags: review+

Comment 10

8 months ago
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f3ad3c9327b9
webdriver, geckodriver: remove elementTap command r=whimboo

Comment 11

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f3ad3c9327b9
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.