Bug 1561435 - Fix linting errors for testing/, r=standard8

# ignore-this-changeset

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

--HG--
extra : source : 7c8fead01ad5b68b81c2dfe3006eb497abb23f44
extra : intermediate-source : c900487402597c47ee28e088af19b4ad786df50f

testing/specialpowers/content/SpecialPowersAPIParent.jsm

@@ -452,8 +452,8 @@ class SpecialPowersAPIParent extends JSWindowActorParent {
    * messageManager callback function
    * This will get requests from our API in the window and process them in chrome for it
    **/
+  // eslint-disable-next-line complexity
   receiveMessage(aMessage) {
-    // eslint-disable-line complexity
     // We explicitly return values in the below code so that this function
     // doesn't trigger a flurry of warnings about "does not always return
     // a value".

testing/talos/talos/pageloader/chrome/lh_fnbpaint.js

@@ -36,6 +36,7 @@ function _contentFNBPaintHandler() {
 
 addEventListener(
   "load",
+  // eslint-disable-next-line no-undef
   contentLoadHandlerCallback(_contentFNBPaintHandler),
   true
-); // eslint-disable-line no-undef
+);

testing/talos/talos/pageloader/chrome/lh_hero.js

@@ -41,6 +41,7 @@ function _contentHeroLoadHandler() {
 
 addEventListener(
   "load",
+  // eslint-disable-next-line no-undef
   contentLoadHandlerCallback(_contentHeroLoadHandler),
   true
-); // eslint-disable-line no-undef
+);

testing/talos/talos/pageloader/chrome/lh_moz.js

@@ -19,6 +19,7 @@ function _contentPaintHandler() {
 
 addEventListener(
   "load",
+  // eslint-disable-next-line no-undef
   contentLoadHandlerCallback(_contentPaintHandler),
   true
-); // eslint-disable-line no-undef
+);

testing/talos/talos/scripts/talos-debug.js

@@ -35,8 +35,8 @@ window.talosDebug = {
 
   median(values) {
     var clone = values.slice(0);
-    // eslint-disable-next-line no-nested-ternary
     var sorted = clone.sort(function(a, b) {
+      // eslint-disable-next-line no-nested-ternary
       return a > b ? 1 : a < b ? -1 : 0;
     });
     var len = values.length;

testing/talos/talos/talos-powers/chrome/talos-powers-content.js

@@ -130,8 +130,9 @@ addEventListener(
     let uniqueMessageId =
       "TalosPowers:ParentExec:" +
       content.document.documentURI +
+      // eslint-disable-next-line mozilla/avoid-Date-timing
       Date.now() +
-      Math.random(); // eslint-disable-line mozilla/avoid-Date-timing
+      Math.random();
 
     // Listener for the reply from the parent process
     addMessageListener("TalosPowers:ParentExec:ReplyMsg", function done(reply) {

testing/talos/talos/tests/tart/addon/content/framescript.js

@@ -29,8 +29,8 @@
         return;
       }
 
-      // eslint-disable-next-line mozilla/avoid-Date-timing
       var uniqueMessageId =
+        // eslint-disable-next-line mozilla/avoid-Date-timing
         TART_PREFIX + content.document.documentURI + Date.now() + Math.random();
 
       addMessageListener(TART_PREFIX + "chrome-exec-reply", function done(

testing/talos/talos/tests/tresize/addon/content/framescript.js

@@ -4,10 +4,10 @@
   addEventListener(
     TRESIZE_PREFIX + "chrome-run-event",
     function(e) {
-      // eslint-disable-next-line mozilla/avoid-Date-timing
       var uniqueMessageId =
         TRESIZE_PREFIX +
         content.document.documentURI +
+        // eslint-disable-next-line mozilla/avoid-Date-timing
         Date.now() +
         Math.random();
 

testing/xpcshell/head.js

@@ -101,8 +101,8 @@ var runningInParent = true;
 try {
   // Don't use Services.appinfo here as it disables replacing appinfo with stubs
   // for test usage.
-  // eslint-disable-next-line mozilla/use-services
   runningInParent =
+    // eslint-disable-next-line mozilla/use-services
     Cc["@mozilla.org/xre/runtime;1"].getService(Ci.nsIXULRuntime).processType ==
     Ci.nsIXULRuntime.PROCESS_TYPE_DEFAULT;
 } catch (e) {}