David Burns :automatedtester Assignee
	Description • 7 years ago

This is a hangover from B2G and nothing appears to use it anymore. Its not a feature I want us supporting moving forward.

	Andreas Tolfsen ❲:ato❳
	Comment 1 • 7 years ago

The import script feature is actually fairly high quality code and well
tested.  I don’t believe it has caused any extra maintenance issues.

I suppose it is not good to accumulate potentially sizeable chunks of JS
in memory, but I would consider it fairly low priority.

	Andreas Tolfsen ❲:ato❳
	Comment 4 • 7 years ago
mozreview-review

Comment on attachment 8872575 [details]
Bug 1368648: Remove importScript functionality from Marionette

https://reviewboard.mozilla.org/r/144100/#review147884

As I said in my earlier comment, I’m sad to see this go as it is quite excellent code, but I suppose all things must come to an end.

::: commit-message-0daf9:3
(Diff revision 2)
> +This removes importScript functionality that was used extensively for B2G testing
> +but is no longer used in tree.

This should be wrapped at ~72 characters.

	Pulsebot
	Comment 6 • 7 years ago

Pushed by dburns@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0710e3c80efa
Remove importScript functionality from Marionette r=ato

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0710e3c80efa