Bug 1793227, part 8 - Manual eslint fixes for js/xpconnect/tests/browser/. r=kmag

browser_exception_leak.js is intentionally creating a leak, so maybe the unused
doc is intentional.

browser_import_mapped_jsm.js is deliberately testing the behavior of Cu.import,
so ignore the failure there.

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

js/xpconnect/tests/browser/browser_exception_leak.js

@@ -36,7 +36,10 @@ add_task(async function test() {
       // Intentionally hold a reference to the console event.
       let leakedConsoleEvent = consoleEvents[0];
 
+      // XXX I think this is intentionally leaking |doc|.
+      // eslint-disable-next-line no-unused-vars
       let doc = content.document;
+
       let promise = TestUtils.topicObserved(
         "inner-window-nuked",
         (subject, data) => {

js/xpconnect/tests/browser/browser_import_mapped_jsm.js

@@ -57,6 +57,7 @@ add_task(async function test_chrome_utils_import() {
 add_task(async function test_cu_import() {
   for (const file of JSMs) {
     try {
+      // eslint-disable-next-line mozilla/use-chromeutils-import
       Cu.import(file, {});
       ok(true, `Imported ${file}`);
     } catch (e) {