From d226fcc86a0395cfec6ccc924c10c975e28cd3cb Mon Sep 17 00:00:00 2001 From: Mark Banner Date: Wed, 11 Apr 2018 16:10:35 +0100 Subject: [PATCH] Bug 1453383 - Enable ESLint for devtools/client/{webaudioeditor,webide}/. r=jdescottes MozReview-Commit-ID: Gm77Z0T3oJq --HG-- extra : rebase_source : 9a23baa102558d30302baf5044da207b7d79f3b6 --- .eslintignore | 16 +++++++-------- devtools/.eslintrc.js | 11 ++++++++++ devtools/client/webaudioeditor/models.js | 1 + .../browser_wa_properties-view-edit-01.js | 6 +++--- .../test/doc_destroy-nodes.html | 20 +++++++++---------- devtools/client/webaudioeditor/test/head.js | 4 ++-- .../client/webaudioeditor/views/inspector.js | 5 ++--- devtools/client/webide/content/details.js | 3 ++- .../webide/content/devicepreferences.js | 4 +++- devtools/client/webide/content/newapp.js | 4 +--- .../client/webide/content/project-panel.js | 1 + .../client/webide/content/runtime-panel.js | 1 + .../client/webide/content/runtimedetails.js | 5 +++-- devtools/client/webide/content/webide.js | 10 ++++++---- devtools/client/webide/content/wifi-auth.js | 1 - devtools/client/webide/modules/addons.js | 1 - devtools/client/webide/modules/app-manager.js | 4 ---- .../client/webide/modules/app-projects.js | 8 ++++---- .../client/webide/modules/app-validator.js | 2 +- devtools/client/webide/modules/config-view.js | 9 ++------- .../client/webide/modules/project-list.js | 4 +--- .../client/webide/modules/runtime-list.js | 4 ---- devtools/client/webide/modules/runtimes.js | 1 - devtools/client/webide/modules/tab-store.js | 2 -- devtools/client/webide/modules/utils.js | 2 +- devtools/client/webide/test/browser_tabs.js | 2 +- .../client/webide/test/device_front_shared.js | 4 +++- devtools/client/webide/test/head.js | 4 ++-- devtools/client/webide/test/test_addons.html | 8 ++++---- .../webide/test/test_app_validator.html | 10 ++++------ .../webide/test/test_device_preferences.html | 1 + .../webide/test/test_manifestUpdate.html | 1 - devtools/client/webide/test/test_runtime.html | 4 +++- devtools/client/webide/test/test_toolbox.html | 2 ++ 34 files changed, 83 insertions(+), 82 deletions(-) diff --git a/.eslintignore b/.eslintignore index e02d302d64f5..750b54835557 100644 --- a/.eslintignore +++ b/.eslintignore @@ -116,17 +116,9 @@ devtools/client/storage/test/*.html !devtools/client/storage/test/storage-search.html !devtools/client/storage/test/storage-unsecured-iframe.html !devtools/client/storage/test/storage-unsecured-iframe-usercontextid.html -devtools/client/webaudioeditor/** -devtools/client/webconsole/old/net/** -!devtools/client/webconsole/test/mochitest/** -devtools/client/webconsole/old/test/** -devtools/client/webconsole/old/webconsole.js -devtools/client/webide/** -!devtools/client/webide/components/webideCli.js devtools/server/tests/browser/storage-*.html !devtools/server/tests/browser/storage-unsecured-iframe.html devtools/server/tests/browser/stylesheets-nested-iframes.html -devtools/server/tests/unit/xpcshell_debugging_script.js devtools/client/shared/webpack/shims/test/test_clipboard.html devtools/shared/qrcode/tests/mochitest/test_decode.html devtools/shared/tests/mochitest/*.html @@ -137,6 +129,10 @@ devtools/client/commandline/** # Soon to be removed, the new/ directory is explicitly excluded below due to # also being an imported repository. devtools/client/debugger/** +# Soon to be removed +devtools/client/webconsole/old/net/** +devtools/client/webconsole/old/test/** +devtools/client/webconsole/old/webconsole.js # Ignore devtools imported repositories devtools/client/debugger/new/** @@ -144,6 +140,7 @@ devtools/client/shared/components/reps/** # Ignore devtools preferences files devtools/client/preferences/** +devtools/client/webide/preferences/** devtools/shared/preferences/** devtools/startup/preferences/devtools-startup.js @@ -180,6 +177,9 @@ devtools/server/tests/unit/babel_and_browserify_script_with_source_map.js devtools/server/tests/unit/setBreakpoint* devtools/server/tests/unit/sourcemapped.js +# devtools specific format test file +devtools/server/tests/unit/xpcshell_debugging_script.js + # dom/ exclusions dom/abort/** dom/animation/** diff --git a/devtools/.eslintrc.js b/devtools/.eslintrc.js index 2ca546f4affc..c0b2b6d3f8d5 100644 --- a/devtools/.eslintrc.js +++ b/devtools/.eslintrc.js @@ -47,6 +47,7 @@ module.exports = { "client/scratchpad/**", "client/shared/*.jsm", "client/shared/widgets/*.jsm", + "client/webide/**", ], "rules": { "consistent-return": "off", @@ -56,6 +57,7 @@ module.exports = { "client/framework/**", "client/scratchpad/**", "client/shared/AppCacheUtils.jsm", + "client/webide/**", ], "rules": { "max-nested-callbacks": "off", @@ -66,6 +68,7 @@ module.exports = { "client/scratchpad/**", "client/shared/*.jsm", "client/shared/widgets/*.jsm", + "client/webide/**", ], "rules": { "max-len": "off", @@ -84,6 +87,7 @@ module.exports = { "client/scratchpad/**", "client/shared/*.jsm", "client/shared/widgets/*.jsm", + "client/webide/**", ], "rules": { "mozilla/no-aArgs": "off", @@ -102,6 +106,7 @@ module.exports = { "client/scratchpad/**", "client/shared/AppCacheUtils.jsm", "client/shared/widgets/*.jsm", + "client/webide/**", ], "rules": { "no-shadow": "off", @@ -110,6 +115,7 @@ module.exports = { "files": [ "client/framework/**", "client/scratchpad/**", + "client/webide/**", ], "rules": { "strict": "off", @@ -122,16 +128,21 @@ module.exports = { // Note: Bug 1342237 may be removing shadereditor, check before // doing more work on enabling these rules. "client/shadereditor/**", + // Note: Bug 1403944 may be removing webaudioeditor, check before + // doing more work on enabling these rules. + "client/webaudioeditor/**", ], "rules": { "consistent-return": "off", "max-len": "off", "mozilla/no-aArgs": "off", "mozilla/var-only-at-top-level": "off", + "no-redeclare": "off", "no-return-assign": "off", "no-shadow": "off", "no-undef": "off", "no-unused-vars": "off", + "no-useless-call": "off", "strict": "off", } }, { diff --git a/devtools/client/webaudioeditor/models.js b/devtools/client/webaudioeditor/models.js index 19930273bf28..046b25e0d897 100644 --- a/devtools/client/webaudioeditor/models.js +++ b/devtools/client/webaudioeditor/models.js @@ -180,6 +180,7 @@ class AudioNodesCollection extends EventEmitter { * @return AudioNodeModel */ add(obj) { + // eslint-disable-next-line new-cap let node = new this.model(obj); node.collection = this; diff --git a/devtools/client/webaudioeditor/test/browser_wa_properties-view-edit-01.js b/devtools/client/webaudioeditor/test/browser_wa_properties-view-edit-01.js index 9ad6442e13f3..ad635e8ae7bc 100644 --- a/devtools/client/webaudioeditor/test/browser_wa_properties-view-edit-01.js +++ b/devtools/client/webaudioeditor/test/browser_wa_properties-view-edit-01.js @@ -37,17 +37,17 @@ add_task(async function() { }, "default loaded string"); click(panelWin, findGraphNode(panelWin, nodeIds[2])); - await waitForInspectorRender(panelWin, EVENTS), + await waitForInspectorRender(panelWin, EVENTS); checkVariableView(gVars, 0, { "gain": 0 }, "default loaded number"); click(panelWin, findGraphNode(panelWin, nodeIds[1])); - await waitForInspectorRender(panelWin, EVENTS), + await waitForInspectorRender(panelWin, EVENTS); await setAndCheck(0, "type", "square", "square", "sets string as string"); click(panelWin, findGraphNode(panelWin, nodeIds[2])); - await waitForInspectorRender(panelWin, EVENTS), + await waitForInspectorRender(panelWin, EVENTS); await setAndCheck(0, "gain", "0.005", 0.005, "sets number as number"); await setAndCheck(0, "gain", "0.1", 0.1, "sets float as float"); await setAndCheck(0, "gain", ".2", 0.2, "sets float without leading zero as float"); diff --git a/devtools/client/webaudioeditor/test/doc_destroy-nodes.html b/devtools/client/webaudioeditor/test/doc_destroy-nodes.html index 0a49eddc4b53..6b1d4b9466d2 100644 --- a/devtools/client/webaudioeditor/test/doc_destroy-nodes.html +++ b/devtools/client/webaudioeditor/test/doc_destroy-nodes.html @@ -17,18 +17,18 @@ // side. var keepAlive = []; (function() { - let ctx = new AudioContext(); - let osc = ctx.createOscillator(); - let gain = ctx.createGain(); + let ctx = new AudioContext(); + let osc = ctx.createOscillator(); + let gain = ctx.createGain(); - for (let i = 0; i < 10; i++) { - keepAlive.push(ctx.createBufferSource()); - } + for (let i = 0; i < 10; i++) { + keepAlive.push(ctx.createBufferSource()); + } - osc.connect(gain); - gain.connect(ctx.destination); - gain.gain.value = 0; - osc.start(); + osc.connect(gain); + gain.connect(ctx.destination); + gain.gain.value = 0; + osc.start(); })(); diff --git a/devtools/client/webaudioeditor/test/head.js b/devtools/client/webaudioeditor/test/head.js index 8e1668b94685..179bbb227733 100644 --- a/devtools/client/webaudioeditor/test/head.js +++ b/devtools/client/webaudioeditor/test/head.js @@ -118,7 +118,7 @@ function getN(front, eventName, count, spread) { let actors = []; info(`Waiting for ${count} ${eventName} events`); - return new Promise((resolve, reject) => { + return new Promise((resolve) => { front.on(eventName, function onEvent(...args) { let actor = args[0]; if (actors.length !== count) { @@ -164,7 +164,7 @@ function waitForGraphRendered(front, nodeCount, edgeCount, paramEdgeCount) { let eventName = front.EVENTS.UI_GRAPH_RENDERED; info(`Wait for graph rendered with ${nodeCount} nodes, ${edgeCount} edges`); - return new Promise((resolve, reject) => { + return new Promise((resolve) => { front.on(eventName, function onGraphRendered(nodes, edges, pEdges) { let paramEdgesDone = paramEdgeCount != null ? paramEdgeCount === pEdges : true; info(`Got graph rendered with ${nodes} / ${nodeCount} nodes, ` + diff --git a/devtools/client/webaudioeditor/views/inspector.js b/devtools/client/webaudioeditor/views/inspector.js index 39817f3184e1..ef0ace829b4a 100644 --- a/devtools/client/webaudioeditor/views/inspector.js +++ b/devtools/client/webaudioeditor/views/inspector.js @@ -85,9 +85,8 @@ var InspectorView = { $("#web-audio-editor-details-pane-empty").removeAttribute("hidden"); $("#web-audio-editor-tabs").setAttribute("hidden", "true"); window.emit(EVENTS.UI_INSPECTOR_NODE_SET, null); - } - // Otherwise load up the tabs view and hide the empty placeholder - else { + } else { + // Otherwise load up the tabs view and hide the empty placeholder $("#web-audio-editor-details-pane-empty").setAttribute("hidden", "true"); $("#web-audio-editor-tabs").removeAttribute("hidden"); this._buildToolbar(); diff --git a/devtools/client/webide/content/details.js b/devtools/client/webide/content/details.js index a92f48fd9b95..82118d3ff626 100644 --- a/devtools/client/webide/content/details.js +++ b/devtools/client/webide/content/details.js @@ -3,7 +3,6 @@ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ const {require} = ChromeUtils.import("resource://devtools/shared/Loader.jsm", {}); -const Services = require("Services"); const {AppManager} = require("devtools/client/webide/modules/app-manager"); window.addEventListener("load", function() { @@ -117,6 +116,8 @@ function updateUI() { AppManager.update("details"); } +// Used in details.xhtml. +/* exported removeProject */ function removeProject() { AppManager.removeSelectedProject(); } diff --git a/devtools/client/webide/content/devicepreferences.js b/devtools/client/webide/content/devicepreferences.js index 667679d0213f..02f5ee79e758 100644 --- a/devtools/client/webide/content/devicepreferences.js +++ b/devtools/client/webide/content/devicepreferences.js @@ -59,7 +59,9 @@ function SearchField(event) { configView.search(event); } -var getAllPrefs; // Used by tests +// Used by tests +/* exported getAllPrefs */ +var getAllPrefs; function BuildUI() { configView.resetTable(); diff --git a/devtools/client/webide/content/newapp.js b/devtools/client/webide/content/newapp.js index e6d2917963df..1bc25f1029fb 100644 --- a/devtools/client/webide/content/newapp.js +++ b/devtools/client/webide/content/newapp.js @@ -5,7 +5,6 @@ "use strict"; const {require} = ChromeUtils.import("resource://devtools/shared/Loader.jsm", {}); -const {XPCOMUtils} = require("resource://gre/modules/XPCOMUtils.jsm"); const Services = require("Services"); const {FileUtils} = require("resource://gre/modules/FileUtils.jsm"); const {AppProjects} = require("devtools/client/webide/modules/app-projects"); @@ -67,8 +66,7 @@ function getTemplatesJSON() { } function failAndBail(msg) { - let promptService = Cc["@mozilla.org/embedcomp/prompt-service;1"].getService(Ci.nsIPromptService); - promptService.alert(window, "error", msg); + Services.prompt.alert(window, "error", msg); window.close(); } diff --git a/devtools/client/webide/content/project-panel.js b/devtools/client/webide/content/project-panel.js index 072e9d0678e6..e10230c2bdf6 100644 --- a/devtools/client/webide/content/project-panel.js +++ b/devtools/client/webide/content/project-panel.js @@ -2,6 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ +/* exported ProjectPanel */ var ProjectPanel = { // TODO: Expand function to save toggle state. toggleSidebar: function() { diff --git a/devtools/client/webide/content/runtime-panel.js b/devtools/client/webide/content/runtime-panel.js index faac629ba02d..6563a0921f6e 100644 --- a/devtools/client/webide/content/runtime-panel.js +++ b/devtools/client/webide/content/runtime-panel.js @@ -2,6 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ +/* exported RuntimePanel */ var RuntimePanel = { // TODO: Expand function to save toggle state. toggleSidebar: function() { diff --git a/devtools/client/webide/content/runtimedetails.js b/devtools/client/webide/content/runtimedetails.js index 7b39766f964b..61bd680a21d9 100644 --- a/devtools/client/webide/content/runtimedetails.js +++ b/devtools/client/webide/content/runtimedetails.js @@ -52,7 +52,9 @@ function generateFields(json) { } } -var getDescriptionPromise; // Used by tests +// Used by tests +/* exported getDescriptionPromise */ +var getDescriptionPromise; function BuildUI() { let table = document.querySelector("table"); table.innerHTML = ""; @@ -70,7 +72,6 @@ function CheckLockState() { let adbCheckResult = document.querySelector("#adb-check > .yesno"); let devtoolsCheckResult = document.querySelector("#devtools-check > .yesno"); let flipCertPerfButton = document.querySelector("#devtools-check button"); - let adbRootButton = document.querySelector("#adb-check button"); let flipCertPerfAction = document.querySelector("#devtools-check > .action"); let adbRootAction = document.querySelector("#adb-check > .action"); diff --git a/devtools/client/webide/content/webide.js b/devtools/client/webide/content/webide.js index 8645c9ffb863..ea2a4b6235f7 100644 --- a/devtools/client/webide/content/webide.js +++ b/devtools/client/webide/content/webide.js @@ -2,6 +2,10 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ +// These files are loaded via webide.xul +/* import-globals-from project-panel.js */ +/* import-globals-from runtime-panel.js */ + const {require} = ChromeUtils.import("resource://devtools/shared/Loader.jsm", {}); const {gDevTools} = require("devtools/client/framework/devtools"); const {gDevToolsBrowser} = require("devtools/client/framework/devtools-browser"); @@ -14,14 +18,12 @@ const EventEmitter = require("devtools/shared/event-emitter"); const promise = require("promise"); const {GetAvailableAddons} = require("devtools/client/webide/modules/addons"); const {getJSON} = require("devtools/client/shared/getjson"); -const utils = require("devtools/client/webide/modules/utils"); const Telemetry = require("devtools/client/shared/telemetry"); const {RuntimeScanners} = require("devtools/client/webide/modules/runtimes"); const {showDoorhanger} = require("devtools/client/shared/doorhanger"); const Strings = Services.strings.createBundle("chrome://devtools/locale/webide.properties"); -const HTML = "http://www.w3.org/1999/xhtml"; const HELP_URL = "https://developer.mozilla.org/docs/Tools/WebIDE/Troubleshooting"; const MAX_ZOOM = 1.4; @@ -435,7 +437,7 @@ var UI = { !this.lastConnectedRuntime) { return; } - let [_, type, id] = this.lastConnectedRuntime.match(/^(\w+):(.+)$/); + let [ , type, id] = this.lastConnectedRuntime.match(/^(\w+):(.+)$/); type = type.toLowerCase(); @@ -658,7 +660,7 @@ var UI = { if (!m) { return; } - let [_, type, project] = m; + let [ , type, project] = m; if (type == "local") { let lastProject = AppProjects.get(project); diff --git a/devtools/client/webide/content/wifi-auth.js b/devtools/client/webide/content/wifi-auth.js index a0a700ae5d90..ee3030410a18 100644 --- a/devtools/client/webide/content/wifi-auth.js +++ b/devtools/client/webide/content/wifi-auth.js @@ -6,7 +6,6 @@ const { require } = ChromeUtils.import("resource://devtools/shared/Loader.jsm", {}); -const Services = require("Services"); const QR = require("devtools/shared/qrcode/index"); window.addEventListener("load", function() { diff --git a/devtools/client/webide/modules/addons.js b/devtools/client/webide/modules/addons.js index b3187dc4b9c9..b4418d054583 100644 --- a/devtools/client/webide/modules/addons.js +++ b/devtools/client/webide/modules/addons.js @@ -6,7 +6,6 @@ const {AddonManager} = require("resource://gre/modules/AddonManager.jsm"); const Services = require("Services"); -const {getJSON} = require("devtools/client/shared/getjson"); const EventEmitter = require("devtools/shared/event-emitter"); var ADB_LINK = Services.prefs.getCharPref("devtools.webide.adbAddonURL"); diff --git a/devtools/client/webide/modules/app-manager.js b/devtools/client/webide/modules/app-manager.js index 873f88f174fc..710ceefff9f1 100644 --- a/devtools/client/webide/modules/app-manager.js +++ b/devtools/client/webide/modules/app-manager.js @@ -2,8 +2,6 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const {Cu} = require("chrome"); - const {TargetFactory} = require("devtools/client/framework/target"); const Services = require("Services"); const {FileUtils} = require("resource://gre/modules/FileUtils.jsm"); @@ -579,8 +577,6 @@ var AppManager = exports.AppManager = { return; } - let installPromise; - if (project.type != "packaged" && project.type != "hosted") { return Promise.reject("Don't know how to install project"); } diff --git a/devtools/client/webide/modules/app-projects.js b/devtools/client/webide/modules/app-projects.js index 636c7aa64291..76687357a207 100644 --- a/devtools/client/webide/modules/app-projects.js +++ b/devtools/client/webide/modules/app-projects.js @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const {Cc, Ci, Cu, Cr} = require("chrome"); +const {Cc, Ci, Cr} = require("chrome"); const EventEmitter = require("devtools/shared/event-emitter"); const {generateUUID} = Cc["@mozilla.org/uuid-generator;1"].getService(Ci.nsIUUIDGenerator); @@ -100,9 +100,9 @@ const IDB = { update: function(project) { return new Promise((resolve, reject) => { - var transaction = IDB._db.transaction(["projects"], "readwrite"); - var objectStore = transaction.objectStore("projects"); - var request = objectStore.put(project); + let transaction = IDB._db.transaction(["projects"], "readwrite"); + let objectStore = transaction.objectStore("projects"); + let request = objectStore.put(project); request.onerror = function(event) { reject("Unable to update project to the AppProjects indexedDB: " + this.error.name + " - " + this.error.message); diff --git a/devtools/client/webide/modules/app-validator.js b/devtools/client/webide/modules/app-validator.js index 499d7f7f1b97..55f81ad2fd69 100644 --- a/devtools/client/webide/modules/app-validator.js +++ b/devtools/client/webide/modules/app-validator.js @@ -3,7 +3,7 @@ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ "use strict"; -var {Ci, Cu} = require("chrome"); +var {Ci} = require("chrome"); const {FileUtils} = require("resource://gre/modules/FileUtils.jsm"); const Services = require("Services"); diff --git a/devtools/client/webide/modules/config-view.js b/devtools/client/webide/modules/config-view.js index 456c50e99c83..285fb24fe578 100644 --- a/devtools/client/webide/modules/config-view.js +++ b/devtools/client/webide/modules/config-view.js @@ -2,8 +2,6 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const {Cu} = require("chrome"); - const EventEmitter = require("devtools/shared/event-emitter"); const Services = require("Services"); const Strings = Services.strings.createBundle("chrome://devtools/locale/webide.properties"); @@ -76,7 +74,7 @@ ConfigView.prototype = { } } } else { - var trs = this._doc.getElementById("device-fields").querySelectorAll("tr"); + let trs = this._doc.getElementById("device-fields").querySelectorAll("tr"); for (let i = 0; i < trs.length; i++) { trs[i].classList.remove("hide"); @@ -164,7 +162,7 @@ ConfigView.prototype = { let table = this._doc.querySelector("table"); let trs = table.querySelectorAll("tr:not(#add-custom-field)"); - for (var i = 0; i < trs.length; i++) { + for (let i = 0; i < trs.length; i++) { table.removeChild(trs[i]); } @@ -299,7 +297,6 @@ ConfigView.prototype = { let table = this._doc.querySelector("table"); let customValueType = table.querySelector("#custom-value-type").value; let customTextEl = table.querySelector("#custom-value-text"); - let customText = customTextEl.value; if (customValueType.length === 0) { return false; @@ -308,10 +305,8 @@ ConfigView.prototype = { switch (customValueType) { case "boolean": customTextEl.type = "checkbox"; - customText = customTextEl.checked; break; case "number": - customText = parseInt(customText, 10) || 0; customTextEl.type = "number"; break; default: diff --git a/devtools/client/webide/modules/project-list.js b/devtools/client/webide/modules/project-list.js index c981d77f71d3..6414075c5838 100644 --- a/devtools/client/webide/modules/project-list.js +++ b/devtools/client/webide/modules/project-list.js @@ -2,8 +2,6 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const {Cu} = require("chrome"); - const Services = require("Services"); const {AppProjects} = require("devtools/client/webide/modules/app-projects"); const {AppManager} = require("devtools/client/webide/modules/app-manager"); @@ -205,7 +203,7 @@ ProjectList.prototype = { let doc = this._doc; let runtimeappsHeaderNode = doc.querySelector("#panel-header-runtimeapps"); let sortedApps = []; - for (let [manifestURL, app] of AppManager.apps) { + for (let [/* manifestURL */, app] of AppManager.apps) { sortedApps.push(app); } sortedApps = sortedApps.sort((a, b) => { diff --git a/devtools/client/webide/modules/runtime-list.js b/devtools/client/webide/modules/runtime-list.js index 801d20d160a8..088b1655f47b 100644 --- a/devtools/client/webide/modules/runtime-list.js +++ b/devtools/client/webide/modules/runtime-list.js @@ -4,14 +4,10 @@ "use strict"; -const Services = require("Services"); const {AppManager} = require("devtools/client/webide/modules/app-manager"); const EventEmitter = require("devtools/shared/event-emitter"); const {RuntimeScanners, WiFiScanner} = require("devtools/client/webide/modules/runtimes"); const {Devices} = require("resource://devtools/shared/apps/Devices.jsm"); -const utils = require("devtools/client/webide/modules/utils"); - -const Strings = Services.strings.createBundle("chrome://devtools/locale/webide.properties"); var RuntimeList; diff --git a/devtools/client/webide/modules/runtimes.js b/devtools/client/webide/modules/runtimes.js index 909533a73d7a..c3a06c9e4c0b 100644 --- a/devtools/client/webide/modules/runtimes.js +++ b/devtools/client/webide/modules/runtimes.js @@ -7,7 +7,6 @@ const {Ci} = require("chrome"); const Services = require("Services"); const {Devices} = require("resource://devtools/shared/apps/Devices.jsm"); -const {Connection} = require("devtools/shared/client/connection-manager"); const {DebuggerServer} = require("devtools/server/main"); const discovery = require("devtools/shared/discovery/discovery"); const EventEmitter = require("devtools/shared/event-emitter"); diff --git a/devtools/client/webide/modules/tab-store.js b/devtools/client/webide/modules/tab-store.js index a1ba9ecea47d..8d85a2e08d8a 100644 --- a/devtools/client/webide/modules/tab-store.js +++ b/devtools/client/webide/modules/tab-store.js @@ -2,8 +2,6 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const { Cu } = require("chrome"); - const { TargetFactory } = require("devtools/client/framework/target"); const EventEmitter = require("devtools/shared/event-emitter"); const { Connection } = require("devtools/shared/client/connection-manager"); diff --git a/devtools/client/webide/modules/utils.js b/devtools/client/webide/modules/utils.js index 494af25330fe..1aa16e0947f4 100644 --- a/devtools/client/webide/modules/utils.js +++ b/devtools/client/webide/modules/utils.js @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -const { Cc, Cu, Ci } = require("chrome"); +const { Cc, Ci } = require("chrome"); const { FileUtils } = require("resource://gre/modules/FileUtils.jsm"); const Services = require("Services"); const Strings = Services.strings.createBundle("chrome://devtools/locale/webide.properties"); diff --git a/devtools/client/webide/test/browser_tabs.js b/devtools/client/webide/test/browser_tabs.js index 740d01abbdb1..71cb5a8a7d3c 100644 --- a/devtools/client/webide/test/browser_tabs.js +++ b/devtools/client/webide/test/browser_tabs.js @@ -6,7 +6,7 @@ const TEST_URI = "http://example.com/browser/devtools/client/webide/test/doc_tab function test() { waitForExplicitFinish(); - requestCompleteLog(); + SimpleTest.requestCompleteLog(); (async function() { // Since we test the connections set below, destroy the server in case it diff --git a/devtools/client/webide/test/device_front_shared.js b/devtools/client/webide/test/device_front_shared.js index 01b99571d138..899480bb469b 100644 --- a/devtools/client/webide/test/device_front_shared.js +++ b/devtools/client/webide/test/device_front_shared.js @@ -1,6 +1,8 @@ /* Any copyright is dedicated to the Public Domain. http://creativecommons.org/publicdomain/zero/1.0/ */ +/* eslint no-unused-vars: ["error", {"args": "none", "vars": "local"}] */ + "use strict"; var customName; @@ -171,7 +173,7 @@ function addNewFieldBoolean() { ok(found, "Found new boolean field line"); // Mouse event trigger - var mouseClick = new MouseEvent("click", { + let mouseClick = new MouseEvent("click", { canBubble: true, cancelable: true, view: doc.parent, diff --git a/devtools/client/webide/test/head.js b/devtools/client/webide/test/head.js index 7e4ef32060fb..1ab82be681a1 100644 --- a/devtools/client/webide/test/head.js +++ b/devtools/client/webide/test/head.js @@ -44,8 +44,8 @@ var openWebIDE = async function(autoInstallAddons) { Services.prefs.setBoolPref("devtools.webide.autoinstallADBHelper", !!autoInstallAddons); - let ww = Cc["@mozilla.org/embedcomp/window-watcher;1"].getService(Ci.nsIWindowWatcher); - let win = ww.openWindow(null, "chrome://webide/content/", "webide", "chrome,centerscreen,resizable", null); + let win = Services.ww.openWindow(null, "chrome://webide/content/", "webide", + "chrome,centerscreen,resizable", null); await new Promise(resolve => { win.addEventListener("load", function() { diff --git a/devtools/client/webide/test/test_addons.html b/devtools/client/webide/test/test_addons.html index 701101af8bfa..3125299af451 100644 --- a/devtools/client/webide/test/test_addons.html +++ b/devtools/client/webide/test/test_addons.html @@ -15,12 +15,13 @@