Bug 1425349 - Fix mozscreenshots logging where multiple log arguments were used. r=johannh draft
authorMatthew Noorenberghe <mozilla@noorenberghe.ca>
Thu, 14 Dec 2017 16:34:39 -0600
changeset 711874 201508c48d2d5fc5816d776d89e9f7fe4eee386e
parent 711873 8ce6c5ea7b6790e477ef88f9e54d2058b0b5c1b5
child 743912 ca084213dd34acffaf92d8f81f53c5b1e87ea9ec
push id93184
push usermozilla@noorenberghe.ca
push dateThu, 14 Dec 2017 22:50:34 +0000
reviewersjohannh
bugs1425349
milestone59.0a1
Bug 1425349 - Fix mozscreenshots logging where multiple log arguments were used. r=johannh MozReview-Commit-ID: LW8vNkBYTGx
browser/tools/mozscreenshots/mozscreenshots/extension/TestRunner.jsm
--- a/browser/tools/mozscreenshots/mozscreenshots/extension/TestRunner.jsm
+++ b/browser/tools/mozscreenshots/mozscreenshots/extension/TestRunner.jsm
@@ -54,32 +54,32 @@ this.TestRunner = {
                    (new Date()).toISOString().replace(/:/g, "-") + "_" + Services.appinfo.OS];
     let screenshotPath = FileUtils.getFile("TmpD", subDirs).path;
 
     const MOZ_UPLOAD_DIR = env.get("MOZ_UPLOAD_DIR");
     if (MOZ_UPLOAD_DIR) {
       screenshotPath = MOZ_UPLOAD_DIR;
     }
 
-    this.mochitestScope.info("Saving screenshots to:", screenshotPath);
+    this.mochitestScope.info(`Saving screenshots to: ${screenshotPath}`);
 
     let screenshotPrefix = Services.appinfo.appBuildID;
     if (jobName) {
       screenshotPrefix += "-" + jobName;
     }
     screenshotPrefix += "_";
     Screenshot.init(screenshotPath, this._extensionPath, screenshotPrefix);
     this._libDir = this._extensionPath.QueryInterface(Ci.nsIFileURL).file.clone();
     this._libDir.append("chrome");
     this._libDir.append("mozscreenshots");
     this._libDir.append("lib");
 
     let sets = this.loadSets(setNames);
 
-    this.mochitestScope.info(sets.length + " sets:", setNames);
+    this.mochitestScope.info(`${sets.length} sets: ${setNames}`);
     this.combos = new LazyProduct(sets);
     this.mochitestScope.info(this.combos.length + " combinations");
 
     this.currentComboIndex = this.completedCombos = 0;
     this._lastCombo = null;
 
     // Setup some prefs
     Services.prefs.setCharPref("browser.aboutHomeSnippets.updateUrl",