Bug 1422915 - Be explicit about remote-active data when shutting down. r?whimboo
When Marionette starts and shuts down it emits the remote-active
system notification. On starting the data is set to true (a boolean),
but on shutdown it is left undefined which defaults to null.
We should use false (boolean) explicitly.
MozReview-Commit-ID: Er0Wn2pZrOi
--- a/testing/marionette/server.js
+++ b/testing/marionette/server.js
@@ -383,17 +383,17 @@ server.TCPListener = class {
logger.debug(`Resetting recommended pref ${k}`);
Preferences.reset(k);
}
this.alteredPrefs.clear();
// Shutdown server socket, and no longer listen for new connections
this.acceptConnections = false;
- Services.obs.notifyObservers(this, NOTIFY_RUNNING);
+ Services.obs.notifyObservers(this, NOTIFY_RUNNING, false);
this.alive = false;
}
onSocketAccepted(serverSocket, clientSocket) {
let input = clientSocket.openInputStream(0, 0, 0);
let output = clientSocket.openOutputStream(0, 0, 0);
let transport = new DebuggerTransport(input, output);
