webkitpy: Rename WebKitFinder to PathFinder

third_party/WebKit/Tools/Scripts/webkitpy/style/checkers/python.py

 # Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org)
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions
 # are met:
 # 1.  Redistributions of source code must retain the above copyright
 #     notice, this list of conditions and the following disclaimer.
 # 2.  Redistributions in binary form must reproduce the above copyright
 #     notice, this list of conditions and the following disclaimer in the
 #     documentation and/or other materials provided with the distribution.
 #
 # THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND
 # ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
 # WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
 # DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR
 # ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
 # DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
 # SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
 # CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
 # OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 """Supports checking WebKit style in Python files."""
 
 import os
 import re
 import sys
 
 
+from webkitpy.common.path_finder import PathFinder
 from webkitpy.common.system.filesystem import FileSystem
 from webkitpy.common.system.executive import Executive
-from webkitpy.common.webkit_finder import WebKitFinder
 from webkitpy.thirdparty import pep8
 
 
 class PythonChecker(object):
     """Processes text lines for checking style."""
 
     def __init__(self, file_path, handle_style_error):
         self._file_path = file_path
         self._handle_style_error = handle_style_error
 
@@ skipping 22 matching lines @@
         pep8_checker.report_error = _pep8_handle_error
         pep8_checker.check_all()
 
     def _check_pylint(self):
         output = self.run_pylint(self._file_path)
         errors = self._parse_pylint_output(output)
         for line_number, category, message in errors:
             self._handle_style_error(line_number, category, 5, message)
 
     def run_pylint(self, path):
-        wkf = WebKitFinder(FileSystem())
+        finder = PathFinder(FileSystem())
         executive = Executive()
         env = os.environ.copy()
         env['PYTHONPATH'] = os.pathsep.join([
-            wkf.path_from_tools_scripts(),
-            wkf.path_from_blink_source('build', 'scripts'),
-            wkf.path_from_tools_scripts('webkitpy', 'thirdparty'),
-            wkf.path_from_blink_source('bindings', 'scripts'),
-            wkf.path_from_chromium_base('build', 'android'),
-            wkf.path_from_chromium_base('third_party', 'catapult', 'devil'),
-            wkf.path_from_chromium_base('third_party', 'pymock'),
+            finder.path_from_tools_scripts(),
+            finder.path_from_blink_source('build', 'scripts'),
+            finder.path_from_tools_scripts('webkitpy', 'thirdparty'),
+            finder.path_from_blink_source('bindings', 'scripts'),
+            finder.path_from_chromium_base('build', 'android'),
+            finder.path_from_chromium_base('third_party', 'catapult', 'devil'),
+            finder.path_from_chromium_base('third_party', 'pymock'),
         ])
         return executive.run_command([
             sys.executable,
-            wkf.path_from_depot_tools_base('pylint.py'),
+            finder.path_from_depot_tools_base('pylint.py'),
             '--output-format=parseable',
-            '--rcfile=' + wkf.path_from_tools_scripts('webkitpy', 'pylintrc'),
+            '--rcfile=' + finder.path_from_tools_scripts('webkitpy', 'pylintrc'),
             path,
         ], env=env, error_handler=executive.ignore_error)
 
     def _parse_pylint_output(self, output):
         # We filter out these messages because they are bugs in pylint that produce false positives.
         # FIXME: Does it make sense to combine these rules with the rules in style/checker.py somehow?
         FALSE_POSITIVES = [
             # possibly http://www.logilab.org/ticket/98613 ?
             "Instance of 'Popen' has no 'poll' member",
             "Instance of 'Popen' has no 'returncode' member",
@@ skipping 15 matching lines @@
 
             line_number = int(match_obj.group(2))
             category_and_method = match_obj.group(3).split(', ')
             category = 'pylint/' + (category_and_method[0])
             if len(category_and_method) > 1:
                 message = '[%s] %s' % (category_and_method[1], match_obj.group(4))
             else:
                 message = match_obj.group(4)
             errors.append((line_number, category, message))
         return errors