1234063 - Always specify sandbox for scripts to be executed

Status: RESOLVED FIXED

(Testing Graveyard :: JSMarionette, defect)

Description

[Andreas Tolfsen ❲:ato❳]

Bug 1123506 is making it possible for injected scripts to be evaluated with lasting side-effects.  The implementation will be true to the WebDriver specification and consequently will make current Marionette consumers unhappy since they rely on a range of non-conforming features.

An experimental approach is to require consumers that need the special behaviour to specify the `sandbox` parameter to executeScript, executeAsyncScript, and executeJSScript.  This will evaluate the script inside a special sandbox that has the same behaviour as before, with all the bells and whistles our internal consumers rely on.

These special features include, but are not limited to, things such as marionetteScriptFinished, waitFor, and simpletest methods.

This approach seems to generally be working quite well for the Python client as can be demonstrated in the try jobs associated with bug 1123506.  I plan to follow the same approach here by always specifying the `sandbox` parameter in the Marionette Node.js client.

This will unfortunately mean that the client won’t be able to use the WebDriver conformatn sandbox; but this should be considered more of an implementation detail as we can fix that retroactively if the need arises in the future.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] andreastt:bug_1234063 > mozilla-b2g:master

Comment 2

[Michael Henretty [:mikehenrty][:mhenretty]]

Comment on attachment 8700413 [details] [review]
[gaia] andreastt:bug_1234063 > mozilla-b2g:master

Looking good to me, and it seems to add more consistency. r=me

Please squash your commits before merging.