Run auto-formatter (autopep8) on webkitpy.

third_party/WebKit/Tools/Scripts/webkitpy/style/checker.py

 # Copyright (C) 2009 Google Inc. All rights reserved.
 # Copyright (C) 2010 Chris Jerdonek (chris.jerdonek@gmail.com)
 # Copyright (C) 2010 ProFUSION embedded systems
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
@@ skipping 89 matching lines @@
     # PEP8 79 character limit, we ignore the 79-character-limit category
     # pep8/E501 for now.
     #
     # FIXME: Consider bringing WebKit's Python code base into conformance
     #        with the 79 character limit, or some higher limit that is
     #        agreeable to the WebKit project.
     '-pep8/E501',
 
     # FIXME: Move the pylint rules from the pylintrc to here. This will
     # also require us to re-work lint-webkitpy to produce the equivalent output.
-    ]
+]
 
 
 # The path-specific filter rules.
 #
 # This list is order sensitive.  Only the first path substring match
 # is used.  See the FilterConfiguration documentation in filter.py
 # for more information on this list.
 #
 # Each string appearing in this nested list should have at least
 # one associated unit test assertion.  These assertions are located,
 # for example, in the test_path_rules_specifier() unit test method of
 # checker_unittest.py.
 _PATH_RULES_SPECIFIER = [
     # Files in these directories are consumers of the WebKit
     # API and therefore do not follow the same header including
     # discipline as WebCore.
 
-    ([# There is no clean way to avoid "yy_*" names used by flex.
-      "Source/core/css/CSSParser-in.cpp"],
+    ([  # There is no clean way to avoid "yy_*" names used by flex.
+        "Source/core/css/CSSParser-in.cpp"],
      ["-readability/naming"]),
 
     # For third-party Python code, keep only the following checks--
     #
     #   No tabs: to avoid having to set the SVN allow-tabs property.
     #   No trailing white space: since this is easy to correct.
     #   No carriage-return line endings: since this is easy to correct.
     #
     (["webkitpy/thirdparty/"],
      ["-",
       "+pep8/W191",  # Tabs
       "+pep8/W291",  # Trailing white space
       "+whitespace/carriage_return"]),
 
-    ([# Jinja templates: files have .cpp or .h extensions, but contain
-      # template code, which can't be handled, so disable tests.
-      "Source/bindings/templates",
-      "Source/build/scripts/templates"],
+    ([  # Jinja templates: files have .cpp or .h extensions, but contain
+        # template code, which can't be handled, so disable tests.
+        "Source/bindings/templates",
+        "Source/build/scripts/templates"],
      ["-"]),
 
-    ([# IDL compiler reference output
-      # Conforming to style significantly increases the complexity of the code
-      # generator and decreases *its* readability, which is of more concern
-      # than style of the machine-generated code itself.
-      "Source/bindings/tests/results"],
+    ([  # IDL compiler reference output
+        # Conforming to style significantly increases the complexity of the code
+        # generator and decreases *its* readability, which is of more concern
+        # than style of the machine-generated code itself.
+        "Source/bindings/tests/results"],
      ["-"]),
 ]
 
 
 _CPP_FILE_EXTENSIONS = [
     'c',
     'cpp',
     'h',
-    ]
+]
 
 _JSON_FILE_EXTENSION = 'json'
 
 _PYTHON_FILE_EXTENSION = 'py'
 
 _TEXT_FILE_EXTENSIONS = [
     'cc',
     'cgi',
     'css',
     'gyp',
     'gypi',
     'html',
     'idl',
     'in',
     'js',
     'mm',
     'php',
     'pl',
     'pm',
     'rb',
     'sh',
     'txt',
     'xhtml',
     'y',
-    ]
+]
 
 _XCODEPROJ_FILE_EXTENSION = 'pbxproj'
 
 _XML_FILE_EXTENSIONS = [
     'vcproj',
     'vsprops',
-    ]
+]
 
 _PNG_FILE_EXTENSION = 'png'
 
 # Files to skip that are less obvious.
 #
 # Some files should be skipped when checking style. For example,
 # WebKit maintains some files in Mozilla style on purpose to ease
 # future merges.
 _SKIPPED_FILES_WITH_WARNING = [
     "Source/WebKit/gtk/tests/",
     # All WebKit*.h files in Source/WebKit2/UIProcess/API/gtk,
     # except those ending in ...Private.h are GTK+ API headers,
     # which differ greatly from WebKit coding style.
     re.compile(r'Source/WebKit2/UIProcess/API/gtk/WebKit(?!.*Private\.h).*\.h$'),
     re.compile(r'Source/WebKit2/WebProcess/InjectedBundle/API/gtk/WebKit(?!.*Private\.h).*\.h$'),
     'Source/WebKit2/UIProcess/API/gtk/webkit2.h',
     'Source/WebKit2/WebProcess/InjectedBundle/API/gtk/webkit-web-extension.h']
 
 # Files to skip that are more common or obvious.
 #
 # This list should be in addition to files with FileType.NONE.  Files
 # with FileType.NONE are automatically skipped without warning.
 _SKIPPED_FILES_WITHOUT_WARNING = [
     "LayoutTests" + os.path.sep,
     "Source/ThirdParty/leveldb" + os.path.sep,
     # Prevents this being recognized as a text file.
     "Source/WebCore/GNUmakefile.features.am.in",
-    ]
+]
 
 # Extensions of files which are allowed to contain carriage returns.
 _CARRIAGE_RETURN_ALLOWED_FILE_EXTENSIONS = [
     'png',
     'vcproj',
     'vsprops',
-    ]
+]
 
 # The maximum number of errors to report per file, per category.
 # If a category is not a key, then it has no maximum.
 _MAX_REPORTS_PER_CATEGORY = {
     "whitespace/carriage_return": 1
 }
 
 
 def _all_categories():
     """Return the set of all categories used by check-webkit-style."""
@@ skipping 29 matching lines @@
 
 
 def check_webkit_style_configuration(options):
     """Return a StyleProcessorConfiguration instance for check-webkit-style.
 
     Args:
       options: A CommandOptionValues instance.
 
     """
     filter_configuration = FilterConfiguration(
-                               base_rules=_BASE_FILTER_RULES,
-                               path_specific=_PATH_RULES_SPECIFIER,
-                               user_rules=options.filter_rules)
+        base_rules=_BASE_FILTER_RULES,
+        path_specific=_PATH_RULES_SPECIFIER,
+        user_rules=options.filter_rules)
 
     return StyleProcessorConfiguration(filter_configuration=filter_configuration,
-               max_reports_per_category=_MAX_REPORTS_PER_CATEGORY,
-               min_confidence=options.min_confidence,
-               output_format=options.output_format,
-               stderr_write=sys.stderr.write)
+                                       max_reports_per_category=_MAX_REPORTS_PER_CATEGORY,
+                                       min_confidence=options.min_confidence,
+                                       output_format=options.output_format,
+                                       stderr_write=sys.stderr.write)
 
 
 def _create_log_handlers(stream):
     """Create and return a default list of logging.Handler instances.
 
     Format WARNING messages and above to display the logging level, and
     messages strictly below WARNING not to display it.
 
     Args:
       stream: See the configure_logging() docstring.
@@ skipping 102 matching lines @@
         return os.path.splitext(file_path)[1].lstrip(".")
 
     def _should_skip_file_path(self, file_path, skip_array_entry):
         match = re.search("\s*png$", file_path)
         if match:
             return False
         if isinstance(skip_array_entry, str):
             if file_path.find(skip_array_entry) >= 0:
                 return True
         elif skip_array_entry.match(file_path):
-                return True
+            return True
         return False
 
     def should_skip_with_warning(self, file_path):
         """Return whether the given file should be skipped with a warning."""
         for skipped_file in _SKIPPED_FILES_WITH_WARNING:
             if self._should_skip_file_path(file_path, skipped_file):
                 return True
         return False
 
     def should_skip_without_warning(self, file_path):
@@ skipping 312 matching lines @@
         checker = self._dispatcher.dispatch(file_path,
                                             style_error_handler,
                                             min_confidence)
 
         if checker is None:
             raise AssertionError("File should not be checked: '%s'" % file_path)
 
         _log.debug("Using class: " + checker.__class__.__name__)
 
         checker.check(lines)