Merge pull request #8954 from MarcelGerber/unused-vars

Remove unused vars

Author: ingorichter

.jshintrc

@@ -13,7 +13,7 @@
     "regexp"        : true,
     "undef"         : true,
     "strict"        : true,
-    "trailing"      : false,
+    "unused"        : "vars",
 
     "asi"           : false,
     "boss"          : false,
@@ -35,15 +35,14 @@
     "proto"         : false,
     "regexdash"     : false,
     "scripturl"     : false,
-    "smarttabs"     : false,
     "shadow"        : false,
     "sub"           : false,
     "supernew"      : false,
     "validthis"     : false,
 
-    "browser"       : true,
+    "browser"       : false,
     "couch"         : false,
-    "devel"         : false,
+    "devel"         : true,
     "dojo"          : false,
     "jquery"        : false,
     "mootools"      : false,
@@ -52,24 +51,20 @@
     "prototypejs"   : false,
     "rhino"         : false,
     "wsh"           : false,
-    
-    "nomen"         : false,
-    "onevar"        : false,
-    "passfail"      : false,
-    "white"         : false,
 
     "maxerr"        : 100,
-    "predef"        : [
-    ],
     "indent"        : 4,
-    "globals"       : [
-        "require",
-        "define",
-        "brackets",
-        "$",
-        "PathUtils",
-        "window",
-        "navigator",
-        "Mustache"
-    ]
-}
+    "globals"       : {
+        "window": false,
+        "document": false,
+        "setTimeout": false,
+        "require": false,
+        "define": false,
+        "brackets": false,
+        "$": false,
+        "PathUtils": false,
+        "window": false,
+        "navigator": false,
+        "Mustache": false
+    }
+}

Gruntfile.js

@@ -27,8 +27,6 @@ module.exports = function (grunt) {
     // load dependencies
     require('load-grunt-tasks')(grunt, {pattern: ['grunt-contrib-*', 'grunt-targethtml', 'grunt-usemin']});
     grunt.loadTasks('tasks');
-
-    var common = require("./tasks/lib/common")(grunt);
 
     // Project configuration.
     grunt.initConfig({

src/LiveDevelopment/Agents/DOMAgent.js

@@ -41,7 +41,6 @@ define(function DOMAgent(require, exports, module) {
     var $exports = $(exports);
 
     var Inspector = require("LiveDevelopment/Inspector/Inspector");
-    var RemoteAgent = require("LiveDevelopment/Agents/RemoteAgent");
     var EditAgent = require("LiveDevelopment/Agents/EditAgent");
     var DOMNode = require("LiveDevelopment/Agents/DOMNode");
     var DOMHelpers = require("LiveDevelopment/Agents/DOMHelpers");
@@ -123,14 +122,6 @@ define(function DOMAgent(require, exports, module) {
         Inspector.DOM.requestChildNodes(node.nodeId);
     }
 
-    /** Resolve a node
-     * @param {DOMNode} node
-     */
-    function resolveNode(node, callback) {
-        console.assert(node.nodeId, "Attempted to resolve node without id");
-        Inspector.DOM.resolveNode(node.nodeId, callback);
-    }
-
     /** Eliminate the query string from a URL
      * @param {string} URL
      */
@@ -332,4 +323,4 @@ define(function DOMAgent(require, exports, module) {
     exports.applyChange = applyChange;
     exports.load = load;
     exports.unload = unload;
-});
+});

src/LiveDevelopment/Agents/DOMHelpers.js

@@ -23,7 +23,7 @@
 
 
 /*jslint vars: true, plusplus: true, devel: true, nomen: true, indent: 4, forin: true, maxerr: 50, regexp: true */
-/*global define, $ */
+/*global define */
 
 /**
  * DOMHelpers is a collection of functions used by the DOMAgent exports `eachNode(src, callback)`
@@ -230,7 +230,6 @@ define(function DOMHelpersModule(require, exports, module) {
     function eachNode(src, callback) {
         var index = 0;
         var text, range, length, payload;
-        var x = 0;
         while (index < src.length) {
 
             // find the next tag
@@ -267,4 +266,4 @@ define(function DOMHelpersModule(require, exports, module) {
     // Export public functions
     exports.extractPayload = extractPayload;
     exports.eachNode = eachNode;
-});
+});

src/LiveDevelopment/Agents/GotoAgent.js

@@ -67,7 +67,6 @@ define(function GotoAgent(require, exports, module) {
      */
     function _makeHTMLTarget(targets, node) {
         if (node.location) {
-            var target = {};
             var url = DOMAgent.url;
             var location = node.location;
             if (node.canHaveChildren()) {
@@ -86,7 +85,6 @@ define(function GotoAgent(require, exports, module) {
      */
     function _makeCSSTarget(targets, rule) {
         if (rule.sourceURL) {
-            var target = {};
             var url = rule.sourceURL;
             url += ":" + rule.style.range.start;
             var name = rule.selectorList.text;
@@ -102,7 +100,6 @@ define(function GotoAgent(require, exports, module) {
     function _makeJSTarget(targets, callFrame) {
         var script = ScriptAgent.scriptWithId(callFrame.location.scriptId);
         if (script && script.url) {
-            var target = {};
             var url = script.url;
             url += ":" + callFrame.location.lineNumber + "," + callFrame.location.columnNumber;
             var name = callFrame.functionName;
@@ -121,7 +118,7 @@ define(function GotoAgent(require, exports, module) {
 
         // get all css rules that apply to the given node
         Inspector.CSS.getMatchedStylesForNode(node.nodeId, function onMatchedStyles(res) {
-            var i, callFrame, name, script, url, rule, targets = [];
+            var i, targets = [];
             _makeHTMLTarget(targets, node);
             for (i in node.trace) {
                 _makeJSTarget(targets, node.trace[i]);
@@ -221,4 +218,4 @@ define(function GotoAgent(require, exports, module) {
     exports.open = open;
     exports.load = load;
     exports.unload = unload;
-});
+});

src/LiveDevelopment/Agents/RemoteAgent.js

@@ -23,7 +23,7 @@
 
 
 /*jslint vars: true, plusplus: true, devel: true, nomen: true, indent: 4, forin: true, maxerr: 50, regexp: true */
-/*global define, $, XMLHttpRequest, window */
+/*global define, $, window */
 
 /**
  * RemoteAgent defines and provides an interface for custom remote functions

src/LiveDevelopment/Agents/RemoteFunctions.js

@@ -23,7 +23,8 @@
 
 
 /*jslint vars: true, plusplus: true, browser: true, nomen: true, indent: 4, forin: true, maxerr: 50, regexp: true */
-/*global define, $, window, navigator, Node, console */
+/*jshint unused: false */
+/*global window, navigator, Node, console */
 /*theseus instrument: false */
 
 /**
@@ -814,4 +815,4 @@ function RemoteFunctions(experimental) {
         "applyDOMEdits"         : applyDOMEdits,
         "getSimpleDOM"          : getSimpleDOM
     };
-}
+}

src/LiveDevelopment/Agents/ScriptAgent.js

@@ -40,15 +40,6 @@ define(function ScriptAgent(require, exports, module) {
     var _idToScript; // id -> script info
     var _insertTrace; // the last recorded trace of a DOM insertion
 
-    /** Add a call stack trace to a node
-     * @param {integer} node id
-     * @param [{Debugger.CallFrame}] call stack
-     */
-    function _addTraceToNode(nodeId, trace) {
-        var node = DOMAgent.nodeWithId(nodeId);
-        node.trace = trace;
-    }
-
     // TODO: should the parameter to this be an ID rather than a URL?
     /** Get the script information for a given url
      * @param {string} url

src/LiveDevelopment/Documents/CSSDocument.js

@@ -176,7 +176,6 @@ define(function CSSDocumentModule(require, exports, module) {
     CSSDocument.prototype.updateHighlight = function () {
         if (Inspector.config.highlight && this.editor) {
             var editor = this.editor,
-                codeMirror = editor._codeMirror,
                 selectors = [];
             _.each(this.editor.getSelections(), function (sel) {
                 var selector = CSSUtils.findSelectorAtDocumentPos(editor, (sel.reversed ? sel.end : sel.start));

src/LiveDevelopment/Documents/CSSPreprocessorDocument.js

@@ -97,7 +97,6 @@ define(function CSSPreprocessorDocumentModule(require, exports, module) {
     CSSPreprocessorDocument.prototype.updateHighlight = function () {
         if (Inspector.config.highlight && this.editor) {
             var editor = this.editor,
-                codeMirror = editor._codeMirror,
                 selectors = [];
             _.each(this.editor.getSelections(), function (sel) {
                 var selector = CSSUtils.findSelectorAtDocumentPos(editor, (sel.reversed ? sel.end : sel.start));