1354203 - (selenium-atoms) [meta] Remove Selenium atoms from Marionette

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, enhancement)

Description

[Andreas Tolfsen ❲:ato❳]

A long-term goal of Marionette is to remove the dependency on Selenium atoms.  This bug tracks the work needed to remove testing/marionette/atom.js from m-c.

There are a few things that block us from removing the JS fragment atoms in their entirety, most notably that the WebDriver WG has decided to depend on the Selenium getText atom for the foreseeable future due to a lack of cross-browser web platform APIs.

Comment 2

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

Andreas, in the meeting last week you mentioned that we cannot remove all the Selenium atoms. Which of those remaining would that be? Currently we have getElementText, isElementEnabled, and isElementDisplayed. I would like to file the remaining bugs for removal. Thanks.

Comment 3

[David Burns :automatedtester]

isElementDisplayed can't be removed as it is in the spec and getElementText may prove tricky as we are waiting on `innerText` moving to be interoperable in browsers.

Comment 4

[Andreas Tolfsen ❲:ato❳]

The dependency tree here is correct.  After I patched Element
Clear last week, the only remaining command we need to fix is Is
Element Enabled which shouldn’t be very hard.  That is tracked in
https://bugzil.la/1354201.

Comment 5

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

Ok, thank you both!

Comment 6

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

(In reply to David Burns :automatedtester from comment #3)

isElementDisplayed can't be removed as it is in the spec and getElementText
may prove tricky as we are waiting on innerText moving to be interoperable
in browsers.

Based on this feedback from David it looks like that we are done with removing the usage of possible Selenium Atom usage. Only these two atoms remain in our tree and in the WebDriver classic specification, and it looks like that we aren't able to get them removed.

Lets discuss today and close if that is correct.

Comment 7

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

We are not going to remove the other atoms. So this work is done.