1392281 - Make action module take window global instead of container

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, enhancement)

Description

[Andreas Tolfsen ❲:ato❳]

Various functions in the action module take a container object that looks like {frame: <WindowProxy global>, shadowRoot: <element>}.  As part of the new window tracking we will get rid of this container concept.

Because the action module is sufficiently self-contained, we can start with removing it from there already.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1392281 - Pass window global to action module instead of container.

Various functions in the action module take a container object such as

	{frame: <WindowProxy global>, shadowRoot: <Element>}

when it in fact never uses the shadowRoot entry.  As part of the new
window tracking we will get rid of the container concept altogether.

Because the action module is sufficiently self-contained, we can start
with removing it from there already.

Review commit: https://reviewboard.mozilla.org/r/170798/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/170798/

Comment 2

[David Burns :automatedtester]

Comment on attachment 8899519 [details]
Bug 1392281 - Pass window global to action module instead of container.

https://reviewboard.mozilla.org/r/170798/#review176328

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b09e4c30ff7c
Pass window global to action module instead of container. r=automatedtester

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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