Bug 719732 - /*global */ comments in Tilt's codebase add more confusion than help; r=rcampbell
authorVictor Porof <vporof@mozilla.com>
Fri, 20 Jan 2012 13:53:31 +0200
changeset 86802 5cac59d4e1e14c47ce2ade41c30278da2de97dd6
parent 86801 e4216a369836bdb7099131e4000aba19ee755d0c
child 86803 4c5e22a03a2552e325015afcc3b76cfa308fc000
push id805
push userakeybl@mozilla.com
push dateWed, 01 Feb 2012 18:17:35 +0000
treeherdermozilla-aurora@6fb3bf232436 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrcampbell
bugs719732
milestone12.0a1
Bug 719732 - /*global */ comments in Tilt's codebase add more confusion than help; r=rcampbell
browser/devtools/tilt/Tilt.jsm
browser/devtools/tilt/TiltGL.jsm
browser/devtools/tilt/TiltMath.jsm
browser/devtools/tilt/TiltUtils.jsm
browser/devtools/tilt/TiltVisualizer.jsm
browser/devtools/tilt/TiltVisualizerStyle.jsm
browser/devtools/tilt/TiltWorkerCrafter.js
browser/devtools/tilt/TiltWorkerPicker.js
browser/devtools/tilt/test/browser_tilt_01_lazy_getter.js
browser/devtools/tilt/test/browser_tilt_02_notifications-seq.js
browser/devtools/tilt/test/browser_tilt_02_notifications.js
browser/devtools/tilt/test/browser_tilt_03_tab_switch.js
browser/devtools/tilt/test/browser_tilt_04_initialization-key.js
browser/devtools/tilt/test/browser_tilt_04_initialization.js
browser/devtools/tilt/test/browser_tilt_05_destruction-esc.js
browser/devtools/tilt/test/browser_tilt_05_destruction-url.js
browser/devtools/tilt/test/browser_tilt_05_destruction.js
browser/devtools/tilt/test/browser_tilt_arcball-reset-typeahead.js
browser/devtools/tilt/test/browser_tilt_arcball-reset.js
browser/devtools/tilt/test/browser_tilt_arcball.js
browser/devtools/tilt/test/browser_tilt_controller.js
browser/devtools/tilt/test/browser_tilt_gl01.js
browser/devtools/tilt/test/browser_tilt_gl02.js
browser/devtools/tilt/test/browser_tilt_gl03.js
browser/devtools/tilt/test/browser_tilt_gl04.js
browser/devtools/tilt/test/browser_tilt_gl05.js
browser/devtools/tilt/test/browser_tilt_gl06.js
browser/devtools/tilt/test/browser_tilt_gl07.js
browser/devtools/tilt/test/browser_tilt_gl08.js
browser/devtools/tilt/test/browser_tilt_math01.js
browser/devtools/tilt/test/browser_tilt_math02.js
browser/devtools/tilt/test/browser_tilt_math03.js
browser/devtools/tilt/test/browser_tilt_math04.js
browser/devtools/tilt/test/browser_tilt_math05.js
browser/devtools/tilt/test/browser_tilt_math06.js
browser/devtools/tilt/test/browser_tilt_math07.js
browser/devtools/tilt/test/browser_tilt_picking.js
browser/devtools/tilt/test/browser_tilt_picking_delete.js
browser/devtools/tilt/test/browser_tilt_picking_highlight01.js
browser/devtools/tilt/test/browser_tilt_picking_highlight02.js
browser/devtools/tilt/test/browser_tilt_picking_highlight03.js
browser/devtools/tilt/test/browser_tilt_utils01.js
browser/devtools/tilt/test/browser_tilt_utils02.js
browser/devtools/tilt/test/browser_tilt_utils03.js
browser/devtools/tilt/test/browser_tilt_utils04.js
browser/devtools/tilt/test/browser_tilt_utils05.js
browser/devtools/tilt/test/browser_tilt_utils06.js
browser/devtools/tilt/test/browser_tilt_visualizer.js
browser/devtools/tilt/test/browser_tilt_zoom.js
browser/devtools/tilt/test/head.js
--- a/browser/devtools/tilt/Tilt.jsm
+++ b/browser/devtools/tilt/Tilt.jsm
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, Services, TiltGL, TiltUtils, TiltVisualizer */
 "use strict";
 
 const Cu = Components.utils;
 
 // Tilt notifications dispatched through the nsIObserverService.
 const TILT_NOTIFICATIONS = {
 
   // Fires when Tilt starts the initialization.
--- a/browser/devtools/tilt/TiltGL.jsm
+++ b/browser/devtools/tilt/TiltGL.jsm
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, Services, TiltMath, TiltUtils, mat4 */
 "use strict";
 
 const Cc = Components.classes;
 const Ci = Components.interfaces;
 const Cu = Components.utils;
 
 const WEBGL_CONTEXT_NAME = "experimental-webgl";
 
--- a/browser/devtools/tilt/TiltMath.jsm
+++ b/browser/devtools/tilt/TiltMath.jsm
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, TiltUtils */
 "use strict";
 
 const Cu = Components.utils;
 
 Cu.import("resource:///modules/devtools/TiltUtils.jsm");
 
 let EXPORTED_SYMBOLS =
   ["EPSILON", "TiltMath", "vec3", "mat3", "mat4", "quat4"];
--- a/browser/devtools/tilt/TiltUtils.jsm
+++ b/browser/devtools/tilt/TiltUtils.jsm
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, Services, XPCOMUtils, LayoutHelpers */
 "use strict";
 
 const Cc = Components.classes;
 const Ci = Components.interfaces;
 const Cu = Components.utils;
 
 Cu.import("resource://gre/modules/Services.jsm");
 Cu.import("resource://gre/modules/XPCOMUtils.jsm");
--- a/browser/devtools/tilt/TiltVisualizer.jsm
+++ b/browser/devtools/tilt/TiltVisualizer.jsm
@@ -31,19 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, Services, ChromeWorker */
-/*global TiltGL, TiltMath, EPSILON, vec3, mat4, quat4, TiltUtils */
 "use strict";
 
 const Cu = Components.utils;
 const Ci = Components.interfaces;
 
 const ELEMENT_MIN_SIZE = 4;
 const INVISIBLE_ELEMENTS = {
   "head": true,
@@ -503,17 +500,17 @@ TiltVisualizer.Presenter.prototype = {
 
   /**
    * Create the combined mesh representing the document visualization by
    * traversing the document & adding a stack for each node that is drawable.
    *
    * @param {Object} aData
    *                 object containing the necessary mesh verts, texcoord etc.
    */
-  setupMesh: function TVP_setupMesh(aData) /*global TiltVisualizerStyle */
+  setupMesh: function TVP_setupMesh(aData)
   {
     let renderer = this.renderer;
 
     // destroy any previously created mesh
     TiltUtils.destroyObject(this.meshStacks);
     TiltUtils.destroyObject(this.meshWireframe);
 
     // if the renderer was destroyed, don't continue setup
--- a/browser/devtools/tilt/TiltVisualizerStyle.jsm
+++ b/browser/devtools/tilt/TiltVisualizerStyle.jsm
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global Components, TiltMath */
 "use strict";
 
 const Cu = Components.utils;
 
 Cu.import("resource:///modules/devtools/TiltMath.jsm");
 
 let EXPORTED_SYMBOLS = ["TiltVisualizerStyle"];
 let rgba = TiltMath.hex2rgba;
--- a/browser/devtools/tilt/TiltWorkerCrafter.js
+++ b/browser/devtools/tilt/TiltWorkerCrafter.js
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global self*/
 "use strict";
 
 const SIXTEEN_OVER_255 = 16 / 255;
 const ONE_OVER_255 = 1 / 255;
 
 /**
  * Given the initialization data (thickness, sizes and information about
  * each DOM node) this worker sends back the arrays representing
--- a/browser/devtools/tilt/TiltWorkerPicker.js
+++ b/browser/devtools/tilt/TiltWorkerPicker.js
@@ -31,18 +31,16 @@
  * under the terms of either the GPL or the LGPL, and not to allow others to
  * use your version of this file under the terms of the MPL, indicate your
  * decision by deleting the provisions above and replace them with the notice
  * and other provisions required by the LGPL or the GPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  ***** END LICENSE BLOCK *****/
-
-/*global self*/
 "use strict";
 
 /**
  * This worker handles picking, given a set of vertices and a ray (calculates
  * the intersection points and offers back information about the closest hit).
  *
  * Used in the TiltVisualization.Presenter object.
  */
--- a/browser/devtools/tilt/test/browser_tilt_01_lazy_getter.js
+++ b/browser/devtools/tilt/test/browser_tilt_01_lazy_getter.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, Tilt */
 "use strict";
 
 function test() {
   ok(Tilt,
     "The Tilt object wasn't got correctly via defineLazyGetter.");
   is(Tilt.chromeWindow, window,
     "The top-level window wasn't saved correctly");
   ok(Tilt.visualizers,
--- a/browser/devtools/tilt/test/browser_tilt_02_notifications-seq.js
+++ b/browser/devtools/tilt/test/browser_tilt_02_notifications-seq.js
@@ -1,15 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, Tilt, EventUtils, InspectorUI */
-/*global INITIALIZING, INITIALIZED, DESTROYING, BEFORE_DESTROYED, DESTROYED */
 "use strict";
 
 let tabEvents = "";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping notifications test because Tilt isn't enabled.");
     return;
--- a/browser/devtools/tilt/test/browser_tilt_02_notifications.js
+++ b/browser/devtools/tilt/test/browser_tilt_02_notifications.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, Tilt, SHOWN, HIDDEN, INITIALIZING, DESTROYING, DESTROYED */
 "use strict";
 
 let tab0, tab1;
 let testStep = -1;
 let tabEvents = "";
 
 function test() {
   if (!isTiltEnabled()) {
--- a/browser/devtools/tilt/test/browser_tilt_03_tab_switch.js
+++ b/browser/devtools/tilt/test/browser_tilt_03_tab_switch.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt, Tilt */
 "use strict";
 
 let tab0, tab1, tab2;
 let testStep = -1;
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping tab switch test because Tilt isn't enabled.");
--- a/browser/devtools/tilt/test/browser_tilt_04_initialization-key.js
+++ b/browser/devtools/tilt/test/browser_tilt_04_initialization-key.js
@@ -1,15 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, Tilt, TiltUtils, TiltVisualizer, InspectorUI */
-/*global Ci, INITIALIZING, DESTROYED, INSPECTOR_OPENED */
 "use strict";
 
 let id;
 let tiltKey;
 let eventType;
 
 function test() {
   if (!isTiltEnabled()) {
--- a/browser/devtools/tilt/test/browser_tilt_04_initialization.js
+++ b/browser/devtools/tilt/test/browser_tilt_04_initialization.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Tilt, TiltUtils, TiltVisualizer */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping initialization test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_05_destruction-esc.js
+++ b/browser/devtools/tilt/test/browser_tilt_05_destruction-esc.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, Tilt, TiltUtils, InspectorUI, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping destruction test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_05_destruction-url.js
+++ b/browser/devtools/tilt/test/browser_tilt_05_destruction-url.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, Tilt, TiltUtils, InspectorUI, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping destruction test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_05_destruction.js
+++ b/browser/devtools/tilt/test/browser_tilt_05_destruction.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, Tilt, TiltUtils, InspectorUI, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping destruction test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_arcball-reset-typeahead.js
+++ b/browser/devtools/tilt/test/browser_tilt_arcball-reset-typeahead.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, waitForExplicitFinish, executeSoon, finish */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, InspectorUI, TiltVisualizer, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping part of the arcball test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
@@ -99,15 +95,15 @@ function performTest(canvas, arcball, ca
 
         EventUtils.synthesizeKey("VK_R", { type: "keydown" });
 
       }, Math.random() * 1000); // leave enough time for transforms to happen
     }, Math.random() * 1000);
   }, Math.random() * 1000);
 }
 
-function cleanup() { /*global gBrowser */
+function cleanup() {
   info("Cleaning up arcball reset test.");
 
   Services.obs.removeObserver(cleanup, DESTROYED);
   gBrowser.removeCurrentTab();
   finish();
 }
--- a/browser/devtools/tilt/test/browser_tilt_arcball-reset.js
+++ b/browser/devtools/tilt/test/browser_tilt_arcball-reset.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, waitForExplicitFinish, executeSoon, finish */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, InspectorUI, TiltVisualizer, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping part of the arcball test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
@@ -97,15 +93,15 @@ function performTest(canvas, arcball, ca
 
         EventUtils.synthesizeKey("VK_R", { type: "keydown" });
 
       }, Math.random() * 1000); // leave enough time for transforms to happen
     }, Math.random() * 1000);
   }, Math.random() * 1000);
 }
 
-function cleanup() { /*global gBrowser */
+function cleanup() {
   info("Cleaning up arcball reset test.");
 
   Services.obs.removeObserver(cleanup, DESTROYED);
   gBrowser.removeCurrentTab();
   finish();
 }
--- a/browser/devtools/tilt/test/browser_tilt_arcball.js
+++ b/browser/devtools/tilt/test/browser_tilt_arcball.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApprox, isApproxVec, vec3, quat4 */
-/*global TiltVisualizer */
 "use strict";
 
 function cloneUpdate(update) {
   return {
     rotation: quat4.create(update.rotation),
     translation: vec3.create(update.translation)
   };
 }
--- a/browser/devtools/tilt/test/browser_tilt_controller.js
+++ b/browser/devtools/tilt/test/browser_tilt_controller.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isEqualVec, isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, vec3, mat4, quat4 */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping controller test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_gl01.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl01.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, isWebGLSupported, createCanvas, TiltGL */
-/*global WebGLRenderingContext, WebGLProgram */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl02.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl02.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl03.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl03.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl04.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl04.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl05.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl05.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl06.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl06.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl07.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl07.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isnot, info, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_gl08.js
+++ b/browser/devtools/tilt/test/browser_tilt_gl08.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isnot, info, isWebGLSupported, createCanvas, TiltGL */
 "use strict";
 
 let isWebGLAvailable;
 
 function onWebGLFail() {
   isWebGLAvailable = false;
 }
 
--- a/browser/devtools/tilt/test/browser_tilt_math01.js
+++ b/browser/devtools/tilt/test/browser_tilt_math01.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApprox, isApproxVec, TiltMath */
 "use strict";
 
 function test() {
   ok(isApprox(TiltMath.radians(30), 0.523598775),
     "The radians() function didn't calculate the value correctly.");
 
   ok(isApprox(TiltMath.degrees(0.5), 28.64788975),
     "The degrees() function didn't calculate the value correctly.");
--- a/browser/devtools/tilt/test/browser_tilt_math02.js
+++ b/browser/devtools/tilt/test/browser_tilt_math02.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApproxVec, vec3, mat4 */
 "use strict";
 
 function test() {
   let v1 = vec3.create();
 
   ok(v1, "Should have created a vector with vec3.create().");
   is(v1.length, 3, "A vec3 should have 3 elements.");
 
--- a/browser/devtools/tilt/test/browser_tilt_math03.js
+++ b/browser/devtools/tilt/test/browser_tilt_math03.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApproxVec, mat3 */
 "use strict";
 
 function test() {
   let m1 = mat3.create();
 
   ok(m1, "Should have created a matrix with mat3.create().");
   is(m1.length, 9, "A mat3 should have 9 elements.");
 
--- a/browser/devtools/tilt/test/browser_tilt_math04.js
+++ b/browser/devtools/tilt/test/browser_tilt_math04.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApprox, isApproxVec, mat4 */
 "use strict";
 
 function test() {
   let m1 = mat4.create();
 
   ok(m1, "Should have created a matrix with mat4.create().");
   is(m1.length, 16, "A mat4 should have 16 elements.");
 
--- a/browser/devtools/tilt/test/browser_tilt_math05.js
+++ b/browser/devtools/tilt/test/browser_tilt_math05.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApproxVec, mat4 */
 "use strict";
 
 function test() {
   let m1 = mat4.create([
     1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16]);
 
   let m2 = mat4.create([
     0, 5, 9, 13, 2, 6, 10, 14, 3, 7, 11, 15, 4, 8, 12, 16]);
--- a/browser/devtools/tilt/test/browser_tilt_math06.js
+++ b/browser/devtools/tilt/test/browser_tilt_math06.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApprox, isApproxVec, quat4 */
 "use strict";
 
 function test() {
   let q1 = quat4.create();
 
   ok(q1, "Should have created a quaternion with quat4.create().");
   is(q1.length, 4, "A quat4 should have 4 elements.");
 
--- a/browser/devtools/tilt/test/browser_tilt_math07.js
+++ b/browser/devtools/tilt/test/browser_tilt_math07.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isApproxVec, quat4 */
 "use strict";
 
 function test() {
   let q1 = quat4.create([1, 2, 3, 4]);
   let q2 = quat4.create([5, 6, 7, 8]);
 
   quat4.multiply(q1, q2);
   ok(isApproxVec(q1, [24, 48, 48, -6]),
--- a/browser/devtools/tilt/test/browser_tilt_picking.js
+++ b/browser/devtools/tilt/test/browser_tilt_picking.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, InspectorUI, DESTROYED */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping picking test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_picking_delete.js
+++ b/browser/devtools/tilt/test/browser_tilt_picking_delete.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, InspectorUI, NODE_REMOVED, DESTROYED */
 "use strict";
 
 let presenter;
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping picking delete test because Tilt isn't enabled.");
     return;
--- a/browser/devtools/tilt/test/browser_tilt_picking_highlight01.js
+++ b/browser/devtools/tilt/test/browser_tilt_picking_highlight01.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, InspectorUI, HIGHLIGHTING, UNHIGHLIGHTING, DESTROYED */
 "use strict";
 
 let presenter;
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping highlight test because Tilt isn't enabled.");
     return;
--- a/browser/devtools/tilt/test/browser_tilt_picking_highlight02.js
+++ b/browser/devtools/tilt/test/browser_tilt_picking_highlight02.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, InspectorUI, HIGHLIGHTING, UNHIGHLIGHTING, DESTROYED */
 "use strict";
 
 let presenter;
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping highlight test because Tilt isn't enabled.");
     return;
--- a/browser/devtools/tilt/test/browser_tilt_picking_highlight03.js
+++ b/browser/devtools/tilt/test/browser_tilt_picking_highlight03.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, InspectorUI, HIGHLIGHTING, UNHIGHLIGHTING, DESTROYED */
 "use strict";
 
 let presenter;
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping highlight test because Tilt isn't enabled.");
     return;
--- a/browser/devtools/tilt/test/browser_tilt_utils01.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils01.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, TiltUtils */
 "use strict";
 
 function test() {
   let prefs = TiltUtils.Preferences;
   ok(prefs, "The TiltUtils.Preferences wasn't found.");
 
   prefs.create("test-pref-bool", "boolean", true);
   prefs.create("test-pref-int", "integer", 42);
--- a/browser/devtools/tilt/test/browser_tilt_utils02.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils02.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, TiltUtils */
 "use strict";
 
 function test() {
   let l10 = TiltUtils.L10n;
   ok(l10, "The TiltUtils.L10n wasn't found.");
 
 
   ok(l10.stringBundle,
--- a/browser/devtools/tilt/test/browser_tilt_utils03.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils03.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, TiltUtils */
 "use strict";
 
 function test() {
   let dom = TiltUtils.DOM;
 
   is(dom.parentNode, null,
     "The parent node should not be initially set.");
 
--- a/browser/devtools/tilt/test/browser_tilt_utils04.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils04.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, gBrowser, TiltUtils */
 "use strict";
 
 function test() {
   let dom = TiltUtils.DOM;
   ok(dom, "The TiltUtils.DOM wasn't found.");
 
 
   is(dom.initCanvas(), null,
--- a/browser/devtools/tilt/test/browser_tilt_utils05.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils05.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, waitForExplicitFinish, finish, gBrowser */
-/*global Cu, LayoutHelpers, TiltUtils */
 "use strict";
 
 Cu.import("resource:///modules/devtools/LayoutHelpers.jsm");
 
 function init(callback) {
   let iframe = gBrowser.ownerDocument.createElement("iframe");
 
   iframe.addEventListener("load", function onLoad() {
@@ -55,17 +52,17 @@ function test() {
       iframe.contentWindow.innerWidth,
       "The content window width wasn't calculated correctly.");
     is(cwDimensions.height - iframe.contentWindow.scrollMaxY,
       iframe.contentWindow.innerHeight,
       "The content window height wasn't calculated correctly.");
 
     let nodeCoordinates = LayoutHelpers.getRect(
       iframe.contentDocument.getElementById("test-div"), iframe.contentWindow);
- 
+
     let frameOffset = LayoutHelpers.getIframeContentOffset(iframe);
     let frameRect = iframe.getBoundingClientRect();
 
     is(nodeCoordinates.top, frameRect.top + frameOffset[0] + 98,
       "The node coordinates top value wasn't calculated correctly.");
     is(nodeCoordinates.left, frameRect.left + frameOffset[1] + 76,
       "The node coordinates left value wasn't calculated correctly.");
     is(nodeCoordinates.width, 123,
--- a/browser/devtools/tilt/test/browser_tilt_utils06.js
+++ b/browser/devtools/tilt/test/browser_tilt_utils06.js
@@ -1,12 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, isnot, gBrowser, TiltUtils */
 "use strict";
 
 let someObject = {
   a: 1,
   func: function()
   {
     this.b = 2;
   }
--- a/browser/devtools/tilt/test/browser_tilt_visualizer.js
+++ b/browser/devtools/tilt/test/browser_tilt_visualizer.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, isApproxVec, isTiltEnabled, isWebGLSupported, gBrowser*/
-/*global TiltVisualizer */
 "use strict";
 
 function test() {
   if (!isTiltEnabled()) {
     info("Skipping notifications test because Tilt isn't enabled.");
     return;
   }
   if (!isWebGLSupported()) {
--- a/browser/devtools/tilt/test/browser_tilt_zoom.js
+++ b/browser/devtools/tilt/test/browser_tilt_zoom.js
@@ -1,14 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global ok, is, info, waitForExplicitFinish, finish, executeSoon, gBrowser */
-/*global isApprox, isTiltEnabled, isWebGLSupported, createTab, createTilt */
-/*global Services, EventUtils, TiltUtils, InspectorUI, DESTROYED */
 "use strict";
 
 const ZOOM = 2;
 const RESIZE = 50;
 
 function test() {
   let random = Math.random() * 10;
 
--- a/browser/devtools/tilt/test/head.js
+++ b/browser/devtools/tilt/test/head.js
@@ -1,13 +1,10 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/*global Services, Components, gBrowser, executeSoon, info */
-/*global InspectorUI, Tilt, TiltGL, EPSILON */
 "use strict";
 
 let tempScope = {};
 Components.utils.import("resource:///modules/devtools/TiltGL.jsm", tempScope);
 Components.utils.import("resource:///modules/devtools/TiltMath.jsm", tempScope);
 Components.utils.import("resource:///modules/devtools/TiltUtils.jsm", tempScope);
 Components.utils.import("resource:///modules/devtools/TiltVisualizer.jsm", tempScope);
 let TiltGL = tempScope.TiltGL;