Closed Bug 1392281 Opened 7 years ago Closed 7 years ago

Make action module take window global instead of container

Categories

(Remote Protocol :: Marionette, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: ato, Assigned: ato)

References

Details

Attachments

(1 file)

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.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Blocks: 1392339
Comment on attachment 8899519 [details]
Bug 1392281 - Pass window global to action module instead of container.

https://reviewboard.mozilla.org/r/170798/#review176328
Attachment #8899519 - Flags: review?(dburns) → review+
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b09e4c30ff7c
Pass window global to action module instead of container. r=automatedtester
https://hg.mozilla.org/mozilla-central/rev/b09e4c30ff7c
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: