1529300 - Remove support for legacy {"ELEMENT": <UUID>} web element objects

Status: RESOLVED FIXED

(Testing :: geckodriver, enhancement)

Description

[Andreas Tolfsen ❲:ato❳]

geckodriver supports legacy {"ELEMENT": <UUID>} Selenium-styled
web element references. These have been removed from most, if not
all, relevant clients and we should also purge it from geckodriver
in preparation of the 1.0 release.

The legacy web element key is defined here:
https://searchfox.org/mozilla-central/rev/93905b660fc99a5d52b683690dd26471daca08c8/testing/geckodriver/src/command.rs#17

And used here:
https://searchfox.org/mozilla-central/source/testing/geckodriver/src/marionette.rs#396

See this for more information working on geckodriver:
https://firefox-source-docs.mozilla.org/testing/geckodriver/

Comment 1

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

This depends on bug 1495063, right?

Comment 2

[Shivam Singhal [ :championshuttler ]]

Hi :whimboo , :ato , I would like to work on it

Thanks

Comment 3

[Shivam Singhal [ :championshuttler ]]

Attachment: Bug 1529300 - Remove support for legacy {ELEMENT: <UUID>} web element objects. r=ato,whimboo

Comment 4

[Andreas Tolfsen ❲:ato❳]

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] (away 02/28 - 03/03) from comment #1)

This depends on bug 1495063, right?

It shouldn’t be, since that is about not using ad-hoc element
serialisation in communication with Marionette, and this is about
the public-facing {"ELEMENT": "…"} for WebDriver.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b62eba3cf3b6
Remove support for legacy {ELEMENT: <UUID>} web element objects. r=ato

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/b62eba3cf3b6