Open Bug 1353696 Opened 5 years ago Updated 4 years ago

Protect __webDriverComplete global from web content introspection


(Testing :: Marionette, enhancement, P2)

Version 3


(Not tracked)


(Reporter: ato, Unassigned)


(Blocks 1 open bug)


This is a follow-up to bug, which makes the unloadHandler safe for web content introspection by cloning it into the sandbox.

Making the same change for the __webDriverComplete callback is harder because it is being used by the simpletest harness.  If we clone it into the content sandbox, we cannot reuse it again as we do in sandbox.createSimpleTest:

> sandbox.createSimpleTest = function (window, harness) {
>   let sb = sandbox.create(window);
>   sb = sandbox.augment(sb, harness);
>   sb[FINISH] = () => sb[COMPLETE](harness.generate_results());
>   return sb;
> };

A relevant test to run is testing/marionette/harness/marionette_harness/tests/unit/
Priority: -- → P3
Blocks: webdriver
Priority: P3 → P2
You need to log in before you can comment on or make changes to this bug.