js_checker.py: Restore smoke tests for linting violations.

tools/web_dev_style/js_checker.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Presubmit script for Chromium JS resources.
 
 See chrome/browser/PRESUBMIT.py
 """
 
 import regex_check
@@ skipping 44 matching lines @@
   def InheritDocCheck(self, i, line):
     """Checks for use of '@inheritDoc' instead of '@override'."""
     return self.RegexCheck(i, line, r"\* (@inheritDoc)",
         "@inheritDoc is deprecated, use @override instead")
 
   def PolymerLocalIdCheck(self, i, line):
     """Checks for use of element.$.localId."""
     return self.RegexCheck(i, line, r"(?<!this)(\.\$)[\[\.]",
         "Please only use this.$.localId, not element.$.localId")
 
-  def RunEsLintChecks(self, affected_js_files):
+  def RunEsLintChecks(self, affected_js_files, format='stylish'):
     """Runs lint checks using ESLint. The ESLint rules being applied are defined
        in the .eslintrc.js configuration file.
     """
     os_path = self.input_api.os_path
 
     try:
       # Import ESLint.
       _HERE_PATH = os_path.dirname(os_path.realpath(__file__))
       _SRC_PATH = os_path.normpath(os_path.join(_HERE_PATH, '..', '..'))
       import sys
       old_sys_path = sys.path[:]
       sys.path.append(os_path.join(_SRC_PATH, 'third_party', 'node'))
       import node, node_modules
     finally:
       sys.path = old_sys_path
 
     # Extract paths to be passed to ESLint.
     affected_js_files_paths = []
     presubmit_path = self.input_api.PresubmitLocalPath()
     for f in affected_js_files:
       affected_js_files_paths.append(
           os_path.relpath(f.AbsoluteLocalPath(), presubmit_path))
 
     output = node.RunNode([
         node_modules.PathToEsLint(),
         '--color',
+        '--format', format,
         '--ignore-pattern \'!.eslintrc.js\'',
         ' '.join(affected_js_files_paths)])
 
     return [self.output_api.PresubmitError(output)] if output else []
 
   def WrapperTypeCheck(self, i, line):
     """Check for wrappers (new String()) instead of builtins (string)."""
     return self.RegexCheck(i, line,
         r"(?:/\*)?\*.*?@(?:param|return|type) ?"     # /** @param/@return/@type
         r"{[^}]*\b(String|Boolean|Number)\b[^}]*}",  # {(Boolean|Number|String)}
@@ skipping 46 matching lines @@
             'Found JavaScript style violations in %s:' %
             f.LocalPath()] + error_lines
         results.append(self.output_api.PresubmitError('\n'.join(error_lines)))
 
     if results:
       results.append(self.output_api.PresubmitNotifyResult(
           'See the JavaScript style guide at '
           'https://chromium.googlesource.com/chromium/src/+/master/styleguide/web/web.md#JavaScript'))
 
     return results