1651648 - Unify locator strategies for chrome and content

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, task, P3)

Description

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

Currently chrome scope only allows a subset of the allowed locator strategies:

https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/testing/marionette/driver.js#96-102
https://searchfox.org/mozilla-central/source/testing/marionette/listener.js#89-98

What's basically missing are the following:

  element.Strategy.Name,
  element.Strategy.LinkText,
  element.Strategy.PartialLinkText,

Given that XUL is gone and we now have XHTML for chrome windows, we should no longer block these strategies.

By doing that the checks if the given strategy is valid only need to be done in driver.js, and can be removed from listener.js.

Maja, would you be interested to work on that bug?

Comment 1

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Attachment: Bug 1651648 - Unify locator strategies for chrome and content

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f4a2f7b515ae
Unify locator strategies for chrome and content r=marionette-reviewers,whimboo

Comment 3

[Atila Butkovits]

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