1202707 - Remove emulator commands from simpletest harness

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Andreas Tolfsen ❲:ato❳]

By not exposing the emulator command functions on the simpletest harness that we inject as globals when the user executes script, we can avoid a bit of tight coupling between simpletest.js and the chrome- and content space.

Instead we can bind the functions and add them directly in the command implementation.

Comment 1

[Andreas Tolfsen ❲:ato❳]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=136a5e6cdebb

Comment 2

[Andreas Tolfsen ❲:ato❳]

review: https://reviewboard.mozilla.org/r/18505

Comment 3

[Andreas Tolfsen ❲:ato❳]

Attachment: MozReview Request: Bug 1202707: Remove emulator commands from simpletest harness

Bug 1202707: Remove emulator commands from simpletest harness

We attach the emulator commands outside of the simpletest harness.
Removing this makes testing/marionette/simpletest.js less dependent
on Marionette.

r=dburns

Comment 4

[David Burns :automatedtester]

Comment on attachment 8658245 [details]
MozReview Request: Bug 1202707: Remove emulator commands from simpletest harness

https://reviewboard.mozilla.org/r/18507/#review16591

::: testing/marionette/simpletest.js:15
(Diff revision 1)
> -                           heartbeatCallback, testName) {
> +    heartbeatCallback, testName) {

please put the white space back

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bebf1050fa4d

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/bebf1050fa4d