Bug 1362421 - Enable eslint on caps/tests/mochitest/browser_checkloaduri.js. r=standard8
authorHemant Singh Patwal <hemantsingh1612@gmail.com>
Fri, 12 May 2017 20:06:39 +0530
changeset 406510 bbe1cf88947a2f1f50957bf972000f27068abae4
parent 406509 dae04df6968031a834aa9e72ba4722f24672f2fd
child 406511 7113866fea4b5dbc784e8ce81cfff69a48da2eeb
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersstandard8
bugs1362421
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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);