Run jshint over all JavaScript files

chrome_test/chrome_test.js

 /*
  * Copyright (c) 2014 The Native Client Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 'use strict';
 
 // Utilities to allow googletest style tests of apps / extensions.
 
@@ skipping 69 matching lines @@
  */
 chrometest.resetExtension = function() {
   var port = chrometest.newTestPort();
   var count = null;
   port.postMessage({'name': 'reset'});
   return port.wait().then(function(msg) {
     ASSERT_EQ('resetReply', msg.name);
     port.disconnect();
     return msg.count;
   });
-}
+};
 
 /**
  * Get a list of all loaded extensions.
  *
  * This exposes the result of chrome.management.getAll for use by tests.
  * @returns {Promise.Array.<ExtensionInfo>}.
  */
 chrometest.getAllExtensions = function() {
   var port = chrometest.newTestPort();
   port.postMessage({'name': 'getAllExtensions'});
@@ skipping 124 matching lines @@
     var r = new XMLHttpRequest();
     r.open('GET', url, false);
     r.onload = function() {
       if (r.readyState == 4) {
         if (r.status == 200) {
           resolve(r.responseText);
         } else {
           reject(r.status);
         }
       }
-    }
+    };
     r.send();
   });
 };
 
 /**
  * Sleep for a duration.
  * @param {float} ms Timeout in milliseconds.
  * @return {Promise} A promise to wait.
  */
 chrometest.sleep = function(ms) {
@@ skipping 27 matching lines @@
   return chrometest.httpGet('/_command?test_count=' + testCount);
 };
 
 /**
  * Notify the test harness that a test has begun.
  * @param {string} name The full name of the test.
  * @return {Promise} A promise to do it.
  */
 chrometest.beginTest_ = function(name) {
   return chrometest.resetExtension().then(function(count) {
-    if (count != 0) {
+    if (count !== 0) {
       throw new Error(
           'Test extension connections from the last test remain active!');
     }
     console.log('[ RUN      ] ' + name);
     chrometest.passed_ = true;
     chrometest.currentTestName_ = name;
     return chrometest.httpGet(
         '/_command?name=' + encodeURIComponent(name) +
         '&start=1');
   }).then(function(result) {
     chrometest.startTime_ = new Date();
   });
 };
 
 /**
  * Notify the test harness that a test has ended.
  * @return {Promise} A promise to do it.
  */
 chrometest.endTest_ = function() {
   return chrometest.resetExtension().then(function(count) {
     EXPECT_EQ(0, count,
               'all connection to the test extension should be closed');
     var endTime = new Date();
     var duration = endTime.getTime() - chrometest.startTime_.getTime();
     duration = chrometest.formatDuration(duration);
     var name = chrometest.currentTestName_;
+    var resultMsg;
+    var result;
     if (chrometest.passed_) {
-      var resultMsg = '      OK';
-      var result = 1;
+      resultMsg = '      OK';
+      result = 1;
     } else {
-      var resultMsg = ' FAILED ';
-      var result = 0;
+      resultMsg = ' FAILED ';
+      result = 0;
     }
     console.log('[ ' + resultMsg + ' ] ' + name + ' (' + duration + ')');
     chrometest.startTime_ = null;
     chrometest.currentTest_ = null;
     chrometest.currentTestName_ = null;
     return chrometest.httpGet(
         '/_command?name=' + encodeURIComponent(name) + '&' +
         'duration=' + encodeURIComponent(duration) + '&' +
         'result=' + result);
   });
@@ skipping 103 matching lines @@
 /**
  * Check if a string matches a googletest style filter.
  * A filter consists of zero or more ':' separated positive wildcard
  * strings, followed optionally by a '-' and zero or more ':' separated
  * negative wildcard strings.
  * @param string filter A googletest style filter string.
  * @param string s A string to match.
  */
 chrometest.filterMatch = function(filter, s) {
   var parts = filter.split('-');
+  var positive;
+  var negative;
   if (parts.length == 1) {
-    var positive = parts[0].split(':');
-    var negative = [];
+    positive = parts[0].split(':');
+    negative = [];
   } else if (parts.length == 2) {
-    var positive = parts[0].split(':');
-    var negative = parts[1].split(':');
+    positive = parts[0].split(':');
+    negative = parts[1].split(':');
   } else {
     // Treat ill-formated filters as non-matches.
     return false;
   }
-  if (positive.length == 1 && positive[0] == '') {
+  if (positive.length == 1 && positive[0] === '') {
     positive = ['*'];
   }
-  if (negative.length == 1 && negative[0] == '') {
+  if (negative.length == 1 && negative[0] === '') {
     negative = [];
   }
   for (var i = 0; i < positive.length; i++) {
     if (!chrometest.wildcardMatch(positive[i], s)) {
       return false;
     }
   }
   for (var i = 0; i < negative.length; i++) {
     if (chrometest.wildcardMatch(negative[i], s)) {
       return false;
@@ skipping 243 matching lines @@
 //   - A global name makes use through multiple tests convenient.
 //   - Using an existing naming convention makes use and intent clear.
 //   - ALL CAPS highlights the testing constructs visually.
 
 
 // TEST Types
 // ----------
 
 /**
  * Register a test case using a fixture class.
- * @param {string} fixtureClass The test fixture class object.
+ * @param {string} FixtureClass The test fixture class object.
  * @param {string} testName The name of the test.
  * @param {function()} testFunc Called to run the test, may return
  *                              a Promise.
  * @param {string} opt_caseName Optional name for the case, otherwise the
- *     fixtureClass class name is used.
+ *     FixtureClass class name is used.
  */
-function TEST_F(fixtureClass, testName, testFunc, opt_caseName) {
-  if (opt_caseName == undefined) {
-    opt_caseName = fixtureClass.name;
+function TEST_F(FixtureClass, testName, testFunc, opt_caseName) {
+  if (opt_caseName === undefined) {
+    opt_caseName = FixtureClass.name;
   }
   var fullName = opt_caseName + '.' + testName;
   chrometest.tests_.push({
     'name': fullName,
     'call': function() {
       return Promise.resolve().then(function() {
         return chrometest.beginTest_(fullName);
       }).then(function() {
-        chrometest.currentTest_ = new fixtureClass();
+        chrometest.currentTest_ = new FixtureClass();
         return Promise.resolve().then(function() {
           return chrometest.currentTest_.setUp();
         }).then(function() {
           return Promise.resolve().then(function() {
             return testFunc.call(chrometest.currentTest_);
           }).catch(function(error) {
             chrometest.fail();
             return chrometest.error(chrometest.formatError(error));
           });
         }).then(function() {
@@ skipping 97 matching lines @@
   chrometest.expect(a > b, 'Expected ' + a + ' > ' + b + ' when ' + context);
 }
 
 function EXPECT_LE(a, b, context) {
   chrometest.expect(a <= b, 'Expected ' + a + ' <= ' + b + ' when ' + context);
 }
 
 function EXPECT_GE(a, b, context) {
   chrometest.expect(a >= b, 'Expected ' + a + ' >= ' + b + ' when ' + context);
 }