Bug 1757397 - [devtools] Enable strict eslint rule in the debugger. r=nchevobbe

Differential Revision: https://phabricator.services.mozilla.com/D139843

devtools/.eslintrc.js

@@ -78,7 +78,7 @@ module.exports = {
       },
     },
     {
-      files: ["client/framework/**", "client/debugger/test/**"],
+      files: ["client/framework/**"],
       rules: {
         strict: "off",
       },

devtools/client/debugger/test/mochitest/.eslintrc.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 // General rule from /.eslintrc.js only accept folders matching **/test*/browser*/
 // where is this folder doesn't match, so manually apply browser test config
 module.exports = {

devtools/client/debugger/test/mochitest/browser_dbg-asm.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-asm.html");
   await reload(dbg);

devtools/client/debugger/test/mochitest/browser_dbg-async-stepping.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Tests async stepping will step over await statements
+
+"use strict";
+
 add_task(async function test() {
   await pushPref("devtools.debugger.features.async-stepping", true);
   const dbg = await initDebugger("doc-async.html", "async.js");

devtools/client/debugger/test/mochitest/browser_dbg-asyncstacks.js

@@ -4,6 +4,8 @@
 
 // Tests that async stacks include the async separator
 
+"use strict";
+
 add_task(async function() {
   pushPref("devtools.debugger.features.async-captured-stacks", true);
   const dbg = await initDebugger("doc-frames-async.html");

devtools/client/debugger/test/mochitest/browser_dbg-audiocontext.js

@@ -5,6 +5,8 @@
 // Test the AudioContext are paused and resume appropriately when using the
 // debugger.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-audiocontext.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-bfcache.js

@@ -2,10 +2,12 @@
  * 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/>. */
 
-PromiseTestUtils.allowMatchingRejectionsGlobally(/Connection closed/);
-
 // Test the debugger when navigating using the BFCache.
 
+"use strict";
+
+PromiseTestUtils.allowMatchingRejectionsGlobally(/Connection closed/);
+
 add_task(async function() {
   info("Run test with bfcacheInParent DISABLED");
   await pushPref("fission.bfcacheInParent", false);

devtools/client/debugger/test/mochitest/browser_dbg-blackbox-all.js

@@ -9,6 +9,8 @@
 // - 'Un/Blackbox files in this directory'
 // - 'Un/Blackbox files outside this directory'
 
+"use strict";
+
 const SOURCE_FILES = {
   nestedSource: "nested-source.js",
   codeReload1: "code_reload_1.js",

devtools/client/debugger/test/mochitest/browser_dbg-blackbox-original.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // This source map does not have source contents, so it's fetched separately
+
+"use strict";
+
 add_task(async function() {
   // NOTE: the CORS call makes the test run times inconsistent
   const dbg = await initDebugger("doc-sourcemaps3.html");

devtools/client/debugger/test/mochitest/browser_dbg-blackbox.js

@@ -5,6 +5,8 @@
 // This test covers all the blackboxing functionality relating to a selected
 // source open in the debugger editor.
 
+"use strict";
+
 requestLongerTimeout(5);
 
 const contextMenuItems = {

devtools/client/debugger/test/mochitest/browser_dbg-breakable-lines.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const testServer = createVersionizedHttpTestServer("sourcemaps-reload");
 const TEST_URL = testServer.urlFor("index.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-breakable-positions.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const testServer = createVersionizedHttpTestServer("sourcemaps-reload");
 const TEST_URL = testServer.urlFor("index.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-breaking-from-console.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 // Tests that `debugger` statements are hit before the debugger even
 // initializes and it properly highlights the right location in the
 // debugger.

devtools/client/debugger/test/mochitest/browser_dbg-breaking.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 // Tests the breakpoints are hit in various situations.
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-breakpoint-skipping-console.js

@@ -7,6 +7,8 @@
  * skip pausing is enabled.
  */
 
+"use strict";
+
 add_task(async function() {
   const toolbox = await initPane("doc-scripts.html", "webconsole", [
     ["devtools.debugger.skip-pausing", true],

devtools/client/debugger/test/mochitest/browser_dbg-breakpoint-skipping.js

@@ -7,6 +7,8 @@
  * invoking functions without pausing.
  */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html");
   await selectSource(dbg, "simple3.js");

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-actions.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 // Tests to see if we can trigger a breakpoint action via the context menu
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple2.js");

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-columns.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple1.js");
   await selectSource(dbg, "long.js");

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-cond-shortcut.js

@@ -5,6 +5,8 @@
 // Test opening conditional panel using keyboard shortcut.
 // Should access the closest breakpoint to a passed in cursorPosition.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "long.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-cond-source-maps.js

@@ -4,6 +4,9 @@
 
 // Confirms that a conditional panel is opened at the
 // correct location in generated files.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-sourcemaps.html", "entry.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-cond.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple2.js");
   await pushPref("devtools.debugger.features.column-breakpoints", true);

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-debugger-statement.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Test enabling and disabling a debugger statement using editor context menu
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-pause-points.html", "pause-points.js");
   await selectSource(dbg, "pause-points.js");

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-duplicate-functions.js

@@ -5,6 +5,9 @@
 // Tests to make sure we do not accidentally slide the breakpoint up to the first
 // function with the same name in the file.
 // TODO: Likely to remove this test when removing the breakpoint sliding functionality
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger(
     "doc-duplicate-functions.html",

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-in-evaled-sources.js

@@ -7,6 +7,9 @@
 // We wrap it in a setTimeout to avoid errors in the webconsole actor which
 // would still be processing the outcome of the evaluation after we destroy
 // the thread actor.
+
+"use strict";
+
 const EVALED_SOURCE_TEXT = `setTimeout(function() {
   debugger;
   console.log("SECOND LINE");

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-list.js

@@ -5,6 +5,8 @@
 // Testing displaying breakpoints in the breakpoints list and the tooltip
 // shows the thread information.
 
+"use strict";
+
 add_task(async function testBreakpointsListForMultipleTargets() {
   const dbg = await initDebugger(
     "doc_dbg-fission-frame-sources.html",

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-popup.js

@@ -4,6 +4,8 @@
 
 // Verify that we hit breakpoints on popups
 
+"use strict";
+
 const TEST_URI = "https://example.org/document-builder.sjs?html=main page";
 const POPUP_URL = `https://example.com/document-builder.sjs?html=${escape(`popup for breakpoints
   <script>

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-reloading-with-source-changes.js

@@ -10,6 +10,8 @@
 // removed in which case the correct behaviours should begin to pass. To see the
 // failures with breakpoints shifting make flag IS_BREAKPOINT_SHIFTING_ENABLED = false
 
+"use strict";
+
 const IS_BREAKPOINT_SHIFTING_ENABLED = true;
 
 const httpServer = createTestHTTPServer();

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-reloading.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Tests breakpoints syncing when reloading
+
+"use strict";
+
 requestLongerTimeout(3);
 
 // Tests that a breakpoint set is correctly synced after reload

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints-sourcemaps-reloading-with-source-changes.js

@@ -7,6 +7,8 @@
  * source content changes of reload.
  */
 
+"use strict";
+
 const testServer = createVersionizedHttpTestServer("sourcemaps-reload");
 const TEST_URL = testServer.urlFor("index.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-breakpoints.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Test enabling and disabling a breakpoint using the check boxes
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple2.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-browser-content-toolbox.js

@@ -6,6 +6,8 @@
  * Tests that the debugger is succesfully loaded in the Browser Content Toolbox.
  */
 
+"use strict";
+
 const {
   gDevToolsBrowser,
 } = require("devtools/client/framework/devtools-browser");

devtools/client/debugger/test/mochitest/browser_dbg-browser-toolbox-unselected-pause.js

@@ -5,6 +5,8 @@
 // Test that the debugger pauses in the multiprocess browser toolbox even when
 // it hasn't been opened.
 
+"use strict";
+
 requestLongerTimeout(4);
 
 /* import-globals-from ../../../framework/browser-toolbox/test/helpers-browser-toolbox.js */

devtools/client/debugger/test/mochitest/browser_dbg-browser-toolbox-workers.js

@@ -5,6 +5,8 @@
 // Test that all kinds of workers show up properly in the multiprocess browser
 // toolbox.
 
+"use strict";
+
 requestLongerTimeout(4);
 
 /* import-globals-from ../../../framework/browser-toolbox/test/helpers-browser-toolbox.js */

devtools/client/debugger/test/mochitest/browser_dbg-call-stack.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 // Ignore strange errors when shutting down.
 PromiseTestUtils.allowMatchingRejectionsGlobally(/No such actor/);
 

devtools/client/debugger/test/mochitest/browser_dbg-chrome-create.js

@@ -6,6 +6,8 @@
  * Tests that a chrome debugger can be created in a new process.
  */
 
+"use strict";
+
 // There are shutdown issues for which multiple rejections are left uncaught.
 // See bug 1018184 for resolving these issues.
 PromiseTestUtils.allowMatchingRejectionsGlobally(/File closed/);

devtools/client/debugger/test/mochitest/browser_dbg-command-click.js

@@ -4,6 +4,9 @@
 
 // This test checks to see if holding down the command key and clicking on function call
 // will jump the debugger to that call.
+
+"use strict";
+
 add_task(async function() {
   await pushPref("devtools.debugger.features.command-click", true);
   info(

devtools/client/debugger/test/mochitest/browser_dbg-console-async.js

@@ -6,6 +6,8 @@
 // console if necessary.  This cleans up the split console pref so
 // it won't pollute other tests.
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
   Services.prefs.setBoolPref(

devtools/client/debugger/test/mochitest/browser_dbg-console-eval.js

@@ -5,6 +5,8 @@
 // Tests that clicking the DOM node button in any ObjectInspect
 // opens the Inspector panel
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple2.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-console-link.js

@@ -5,6 +5,8 @@
 // Tests opening the console first, clicking a link
 // opens the editor at the correct location.
 
+"use strict";
+
 add_task(async function() {
   const toolbox = await initPane("doc-script-switching.html", "webconsole");
   const node = await waitForLink(toolbox, "hi");

devtools/client/debugger/test/mochitest/browser_dbg-console-map-bindings.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
   const dbg = await initDebugger("doc-strict.html");

devtools/client/debugger/test/mochitest/browser_dbg-console.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
   const dbg = await initDebugger(

devtools/client/debugger/test/mochitest/browser_dbg-content-script-sources.js

@@ -4,6 +4,8 @@
 
 // Tests that the content scripts are listed in the source tree.
 
+"use strict";
+
 add_task(async function() {
   await pushPref("devtools.chrome.enabled", true);
   const extension = await installAndStartExtension();

devtools/client/debugger/test/mochitest/browser_dbg-continue-to-here-click.js

@@ -4,6 +4,8 @@
 
 // Test cmd+click continuing to a line
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-pause-points.html", "pause-points.js");
   await selectSource(dbg, "pause-points.js");

devtools/client/debugger/test/mochitest/browser_dbg-continue-to-here.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-pause-points.html", "pause-points.js");
   await selectSource(dbg, "pause-points.js");

devtools/client/debugger/test/mochitest/browser_dbg-debug-line.js

@@ -4,6 +4,9 @@
 
 // Test ensures zombie debug lines do not persist
 // https://github.com/firefox-devtools/debugger/issues/7755
+
+"use strict";
+
 add_task(async function() {
   // Load test files
   const dbg = await initDebugger("doc-sources.html");

devtools/client/debugger/test/mochitest/browser_dbg-debugger-buttons.js

@@ -10,6 +10,9 @@
  *  4. stepOver to the end of a function
  *  5. stepUp at the end of a function
  */
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-debugger-statements.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-dom-mutation-breakpoints-fission.js

@@ -4,9 +4,10 @@
 
 // Tests dom mutation breakpoints with a remote frame.
 
-/* import-globals-from ../../../inspector/test/shared-head.js */
+"use strict";
 
 // Import helpers for the inspector
+/* import-globals-from ../../../inspector/test/shared-head.js */
 Services.scriptloader.loadSubScript(
   "chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js",
   this

devtools/client/debugger/test/mochitest/browser_dbg-dom-mutation-breakpoints.js

@@ -4,9 +4,10 @@
 
 // Tests adding, disble/enable, and removal of dom mutation breakpoints
 
-/* import-globals-from ../../../inspector/test/shared-head.js */
+"use strict";
 
 // Import helpers for the inspector
+/* import-globals-from ../../../inspector/test/shared-head.js */
 Services.scriptloader.loadSubScript(
   "chrome://mochitests/content/browser/devtools/client/inspector/test/shared-head.js",
   this

devtools/client/debugger/test/mochitest/browser_dbg-eager-eval-skip-pause.js

@@ -4,6 +4,8 @@
 
 // Test that eager evaluation skips breakpoints and debugger statements
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-strict.html");
   const { hud } = await getDebuggerSplitConsole(dbg);

devtools/client/debugger/test/mochitest/browser_dbg-editor-gutter.js

@@ -5,6 +5,8 @@
 // Tests the breakpoint gutter and making sure breakpoint icons exist
 // correctly
 
+"use strict";
+
 // FIXME bug 1524374 removing breakpoints in this test can cause uncaught
 // rejections and make bug 1512742 permafail.
 PromiseTestUtils.allowMatchingRejectionsGlobally(/NS_ERROR_NOT_INITIALIZED/);

devtools/client/debugger/test/mochitest/browser_dbg-editor-highlight.js

@@ -6,6 +6,8 @@
 // matter if the source text doesn't exist yet or even if the source
 // doesn't exist.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "long.js");
   const {

devtools/client/debugger/test/mochitest/browser_dbg-editor-mode.js

@@ -4,6 +4,9 @@
 
 // Tests that the editor sets the correct mode for different file
 // types
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple1.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-editor-scroll.js

@@ -4,6 +4,9 @@
 
 // Tests that the editor keeps proper scroll position per document
 // while also moving to the correct location upon pause/breakpoint selection
+
+"use strict";
+
 requestLongerTimeout(2);
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-editor-select.js

@@ -4,6 +4,9 @@
 
 // Tests that the editor highlights the correct location when the
 // debugger pauses
+
+"use strict";
+
 requestLongerTimeout(2);
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-ember-quickstart.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("ember/quickstart/dist/");
   dbg.actions.toggleMapScopes();

devtools/client/debugger/test/mochitest/browser_dbg-eval-throw.js

@@ -4,6 +4,9 @@
 
 // Test that exceptions thrown while evaluating code will pause at the point the
 // exception was generated when pausing on uncaught exceptions.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-eval-throw.html");
   await togglePauseOnExceptions(dbg, true, true);

devtools/client/debugger/test/mochitest/browser_dbg-event-breakpoints-fission.js

@@ -4,6 +4,8 @@
 
 // Tests early event breakpoints and event breakpoints in a remote frame.
 
+"use strict";
+
 add_task(async function() {
   await pushPref(
     "devtools.debugger.features.event-listeners-breakpoints",

devtools/client/debugger/test/mochitest/browser_dbg-event-breakpoints.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   await pushPref(
     "devtools.debugger.features.event-listeners-breakpoints",

devtools/client/debugger/test/mochitest/browser_dbg-event-handler.js

@@ -5,6 +5,9 @@
 // Test that pausing within an event handler on an element does *not* show the
 // HTML page containing that element. It should show a sources tab containing
 // just the handler's text instead.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-event-handler.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-expressions-error.js

@@ -10,6 +10,8 @@
  * 3. expand properties
  */
 
+"use strict";
+
 const EXPRESSION_SELECTORS = {
   plusIcon: ".watch-expressions-pane button.plus",
   input: "input.input-expression",

devtools/client/debugger/test/mochitest/browser_dbg-expressions-focus.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Ensures the input is displayed and focused when "+" is clicked
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-script-switching.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-expressions-thread.js

@@ -6,6 +6,8 @@
  * Test the watch expressions update when selecting a different thread in the thread panel.
  */
 
+"use strict";
+
 const TEST_COM_URI = `${URL_ROOT_COM}examples/doc_dbg-fission-frame-sources.html`;
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-expressions-watch.js

@@ -10,6 +10,8 @@
  * - disappears when all expressions are removed
  */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-script-switching.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-expressions.js

@@ -10,6 +10,8 @@
  * 4. expanding properties when not paused
  */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-script-switching.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-fission-frame-breakpoint.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const TEST_COM_URI = `${URL_ROOT_COM_SSL}examples/doc_dbg-fission-frame-sources.html`;
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-fission-frame-pause-exceptions.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const TEST_COM_URI = `${URL_ROOT_COM_SSL}examples/doc_dbg-fission-pause-exceptions.html`;
 // Tests Pause on exceptions in remote iframes
 

devtools/client/debugger/test/mochitest/browser_dbg-fission-frame-sources.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const TEST_COM_URI = `${URL_ROOT_COM_SSL}examples/doc_dbg-fission-frame-sources.html`;
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-fission-project-search.js

@@ -2,9 +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/>. */
 
-/* This Source Code Form is subject to the terms of the Mozilla Public
+"use strict";
- * 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 TEST_COM_URI = `${URL_ROOT_COM_SSL}examples/doc_dbg-fission-frame-sources.html`;
 

devtools/client/debugger/test/mochitest/browser_dbg-fission-switch-target.js

@@ -4,6 +4,8 @@
 
 // Test switching for the top-level target.
 
+"use strict";
+
 const PARENT_PROCESS_URI = "about:robots";
 
 add_task(async function() {

devtools/client/debugger/test/mochitest/browser_dbg-gc-breakpoint-positions.js

@@ -4,6 +4,8 @@
 
 // Test that we can set breakpoints in scripts that have been GCed.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger(
     "doc-gc-breakpoint-positions.html",

devtools/client/debugger/test/mochitest/browser_dbg-gc-sources.js

@@ -7,6 +7,8 @@
 // generated and original sources of scripts with source maps specified either
 // inline  or in their HTTP response headers.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger(
     "doc-gc-sources.html",

devtools/client/debugger/test/mochitest/browser_dbg-go-to-line.js

@@ -4,6 +4,8 @@
 
 // Test the "go to line" feature correctly responses to keyboard shortcuts.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "long.js");
   await selectSource(dbg, "long.js");

devtools/client/debugger/test/mochitest/browser_dbg-highlights-calls.js

@@ -4,6 +4,9 @@
 
 // This test checks to see if command button highlights and unhighlights
 // calls when debugger is paused.
+
+"use strict";
+
 add_task(async function() {
   await pushPref("devtools.debugger.features.command-click", true);
   const dbg = await initDebugger("doc-command-click.html", "simple4.js");

devtools/client/debugger/test/mochitest/browser_dbg-html-breakpoints.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-html-breakpoints.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-idb-run-to-completion.js

@@ -4,6 +4,9 @@
 
 // Test that IDB transactions are not processed at microtask checkpoints
 // introduced by debugger hooks.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-idb-run-to-completion.html");
   invokeInTab("test", "doc-xhr-run-to-completion.html");

devtools/client/debugger/test/mochitest/browser_dbg-iframes.js

@@ -4,6 +4,9 @@
 
 // Test is taking too much time to complete on some hardware since
 // release at https://bugzilla.mozilla.org/show_bug.cgi?id=1423158
+
+"use strict";
+
 requestLongerTimeout(3);
 
 /**

devtools/client/debugger/test/mochitest/browser_dbg-inline-cache.js

@@ -9,6 +9,8 @@
  *   - Reload inside debugger with toolbox caching enabled
  */
 
+"use strict";
+
 // Breakpoint position calculations can throw when interrupted by a navigation.
 PromiseTestUtils.allowMatchingRejectionsGlobally(/Resource .*? does not exist/);
 

devtools/client/debugger/test/mochitest/browser_dbg-inline-exceptions.js

@@ -5,6 +5,8 @@
 // This test checks the appearance of an inline exception
 // and the content of the exception tooltip.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-exceptions.html");
   await selectSource(dbg, "exceptions.js");

devtools/client/debugger/test/mochitest/browser_dbg-inline-preview.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Test checking inline preview feature
+
+"use strict";
+
 add_task(async function() {
   await pushPref("devtools.debugger.features.inline-preview", true);
 

devtools/client/debugger/test/mochitest/browser_dbg-inline-script-offset.js

@@ -4,6 +4,8 @@
 
 // Test that breakpoints work when set in inline scripts that do not start at column 0.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-inline-script-offset.html");
   await selectSource(dbg, "doc-inline-script-offset.html");

devtools/client/debugger/test/mochitest/browser_dbg-inspector-integration.js

@@ -5,6 +5,8 @@
 // Tests that clicking the DOM node button in any ObjectInspect
 // opens the Inspector panel
 
+"use strict";
+
 add_task(async function() {
   // Ensures the end panel is wide enough to show the inspector icon
   await pushPref("devtools.debugger.end-panel-size", 600);

devtools/client/debugger/test/mochitest/browser_dbg-keyboard-navigation.js

@@ -4,6 +4,8 @@
 
 // Tests that keyboard navigation into and out of debugger code editor
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple2.js");
   const doc = dbg.win.document;

devtools/client/debugger/test/mochitest/browser_dbg-keyboard-shortcuts-modal.js

@@ -6,6 +6,8 @@
  * Test the keyboard shortcuts modal
  */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebuggerWithAbsoluteURL(
     "data:text/html,Test keyboard shortcuts modal"

devtools/client/debugger/test/mochitest/browser_dbg-keyboard-shortcuts.js

@@ -6,6 +6,8 @@
  * Test keyboard shortcuts.
  */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-debugger-statements.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-layout-changes.js

@@ -6,6 +6,9 @@
  * This if the debugger's layout is correctly modified when the toolbox's
  * host changes.
  */
+
+"use strict";
+
 requestLongerTimeout(2);
 
 let gDefaultHostType = Services.prefs.getCharPref("devtools.toolbox.host");

devtools/client/debugger/test/mochitest/browser_dbg-link-reload.js

@@ -6,6 +6,9 @@
  * Test reload via an href link, which refers to the same document.
  * It seems to cause different codepath compared to F5.
  */
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger(
     "doc-reload-link.html",

devtools/client/debugger/test/mochitest/browser_dbg-log-events.js

@@ -6,6 +6,8 @@
  * Tests that we can log event listeners calls
  */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
   const dbg = await initDebugger(

devtools/client/debugger/test/mochitest/browser_dbg-log-point-mapping.js

@@ -6,6 +6,8 @@
  * Tests that expressions in log points are source mapped.
  */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
 

devtools/client/debugger/test/mochitest/browser_dbg-log-points-workers.js

@@ -6,6 +6,8 @@
  * Tests that log points in a worker are correctly logged to the console
  */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
 

devtools/client/debugger/test/mochitest/browser_dbg-log-points.js

@@ -6,6 +6,8 @@
  * Tests that log points are correctly logged to the console
  */
 
+"use strict";
+
 add_task(async function() {
   Services.prefs.setBoolPref("devtools.toolbox.splitconsoleEnabled", true);
   const dbg = await initDebugger(

devtools/client/debugger/test/mochitest/browser_dbg-many-breakpoints-same-line.js

@@ -5,6 +5,8 @@
 // Test settings multiple types of breakpoints on the same line
 // Only the last should be used
 
+"use strict";
+
 // Line where we set a breakpoint in simple2.js
 const BREAKPOINT_LINE = 5;
 

devtools/client/debugger/test/mochitest/browser_dbg-merge-scopes.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Test that adjacent scopes are merged together as expected.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-merge-scopes.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-message-run-to-completion.js

@@ -4,6 +4,9 @@
 
 // Test that messages from postMessage calls are not delivered while paused in
 // the debugger.
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-message-run-to-completion.html");
   invokeInTab("test", "doc-message-run-to-completion.html");

devtools/client/debugger/test/mochitest/browser_dbg-minified.js

@@ -4,6 +4,8 @@
 
 // Tests minfied + source maps.
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-minified2.html", "sum.js");
   dbg.actions.toggleMapScopes();

devtools/client/debugger/test/mochitest/browser_dbg-navigation-when-paused.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-navigation-when-paused.html");
 

devtools/client/debugger/test/mochitest/browser_dbg-navigation.js

@@ -2,6 +2,8 @@
  * 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/>. */
 
+"use strict";
+
 const SOURCES = [
   "simple1.js",
   "simple2.js",

devtools/client/debugger/test/mochitest/browser_dbg-old-breakpoint.js

@@ -4,6 +4,9 @@
 
 // Test that we show a breakpoint in the UI when there is an old pending
 // breakpoint with an invalid original location.
+
+"use strict";
+
 add_task(async function() {
   clearDebuggerPreferences();
 

devtools/client/debugger/test/mochitest/browser_dbg-outline-filter.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Tests the outline pane fuzzy filtering of outline items
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "long.js");
   await selectSource(dbg, "long.js", 1);

devtools/client/debugger/test/mochitest/browser_dbg-outline-focus.js

@@ -4,6 +4,8 @@
 
 // Tests that after clicking a function in edtior, outline focuses that function
 
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-sources.html", "long.js");
 

devtools/client/debugger/test/mochitest/browser_dbg-outline-pretty.js

@@ -3,6 +3,9 @@
  * file, You can obtain one at <http://mozilla.org/MPL/2.0/>. */
 
 // Tests that the length of outline functions for original and pretty printed source matches
+
+"use strict";
+
 add_task(async function() {
   const dbg = await initDebugger("doc-scripts.html", "simple1.js");