1504479 - Drop second unwrapping argument to ChromeUtils.import

Status: RESOLVED INVALID

(Remote Protocol :: Marionette, enhancement, P1)

Description

[Andreas Tolfsen ❲:ato❳]

When we were previously using Cu.import, we had to pass a second
argument {} to which to temporarily assign the imported object.  We
would then destructure this to just the symbols we’d need:

> const {Foo} = Cu.import("somemodule.js", {});

With ChromeUtils.import we can drop the second argument, which looks
more similar to how ES modules work:

> const {Foo} = Cu.import("somemodule.js");

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: bug 1504479: marionette: drop assignment argument to ChromeUtils;

The second assignment argument was previously needed with Cu.import
as a way to temporarily assign an imported object so only certain
symbols on it would be imported into the scope.  With ChromeUtils.import
the second argument is no longer required.

Comment 2

[Andreas Tolfsen ❲:ato❳]

OK, according to the Phabricator linter I was wrong about this.