JSLint's browser option is true if no environment is defined

Author: dangoor

src/extensions/default/JSLint/main.js

@@ -46,6 +46,9 @@ define(function (require, exports, module) {
             CodeInspection.requestRun(Strings.JSLINT_NAME);
         });
 
+    // Predefined environments understood by JSLint.
+    var ENVIRONMENTS = ["browser", "node", "couch", "rhino"];
+    
     /**
      * Run JSLint on the current document. Reports results to the main UI. Displays
      * a gold star when no errors are found.
@@ -75,6 +78,15 @@ define(function (require, exports, module) {
             options.indent = PreferencesManager.get("spaceUnits");
         }
 
+        // If the user has not defined the environment, we use browser by default.
+        var hasEnvironment = _.some(ENVIRONMENTS, function (env) {
+            return options[env] !== undefined;
+        });
+        
+        if (!hasEnvironment) {
+            options.browser = true;
+        }
+        
         var jslintResult = JSLINT(text, options);
 
         if (!jslintResult) {

src/extensions/default/JSLint/unittest-files/no-errors.js

@@ -1,3 +1,4 @@
 function foo() {
     "use strict";
+    var body = document.body;
 }