Bug 1362421 - Enable eslint on caps/tests/mochitest/browser_checkloaduri.js. r?standard8 draft
authorHemant Singh Patwal <hemantsingh1612@gmail.com>
Fri, 12 May 2017 20:06:39 +0530
changeset 576948 10b4522cd969ab87d1ee5351145168eac8755e2a
parent 576947 1ec1d88637208098802e57d13db9033902e5a229
child 628366 a2e670344e255edc3051d23e562c3ce5e74985dd
push id58542
push userbmo:hemantsingh1612@gmail.com
push dateFri, 12 May 2017 14:37:12 +0000
reviewersstandard8
bugs1362421
milestone55.0a1
Bug 1362421 - Enable eslint on caps/tests/mochitest/browser_checkloaduri.js. r?standard8 MozReview-Commit-ID: ZIec0SdBaa
.eslintignore
caps/tests/mochitest/browser_checkloaduri.js
--- a/.eslintignore
+++ b/.eslintignore
@@ -73,20 +73,17 @@ browser/extensions/pdfjs/content/web**
 browser/extensions/pocket/content/panels/js/tmpl.js
 browser/extensions/pocket/content/panels/js/vendor/**
 browser/locales/**
 # generated or library files in activity-stream
 browser/extensions/activity-stream/data/content/activity-stream.bundle.js
 browser/extensions/activity-stream/vendor/**
 # imported from chromium
 browser/extensions/mortar/**
-
-# caps/ exclusions
-caps/tests/mochitest/browser_checkloaduri.js
-
+
 # devtools/ exclusions
 devtools/client/canvasdebugger/**
 devtools/client/commandline/**
 devtools/client/debugger/**
 devtools/client/framework/**
 !devtools/client/framework/devtools.js
 !devtools/client/framework/devtools-browser.js
 !devtools/client/framework/selection.js
--- a/caps/tests/mochitest/browser_checkloaduri.js
+++ b/caps/tests/mochitest/browser_checkloaduri.js
@@ -256,19 +256,23 @@ add_task(function* () {
   }
 
   // Now test blob URIs, which we need to do in-content.
   yield BrowserTestUtils.withNewTab("http://www.example.com/", function* (browser) {
     yield ContentTask.spawn(
       browser,
       testURL.toString(),
       function* (testURLFn) {
+        // eslint-disable-next-line no-shadow , no-eval
         let testURL = eval("(" + testURLFn + ")");
+        // eslint-disable-next-line no-shadow
         let ssm = Services.scriptSecurityManager;
+        // eslint-disable-next-line no-shadow
         let baseFlags = ssm.STANDARD | ssm.DONT_REPORT_ERRORS;
+        // eslint-disable-next-line no-unused-vars
         let makeURI = Cu.import("resource://gre/modules/BrowserUtils.jsm", {}).BrowserUtils.makeURI;
         let b = new content.Blob(["I am a blob"]);
         let contentBlobURI = content.URL.createObjectURL(b);
         let contentPrincipal = content.document.nodePrincipal;
         // Loading this blob URI from the content page should work:
         testURL(contentPrincipal, contentBlobURI, true, true, true, baseFlags);
         testURL(contentPrincipal, contentBlobURI, true, true, true,
                 baseFlags | ssm.DISALLOW_INHERIT_PRINCIPAL);