Reland of PyLint fixes for tools-webrtc and webrtc/tools

tools-webrtc/check_package_boundaries.py

 #!/usr/bin/env python
 
 # Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
@@ skipping 21 matching lines @@
 ERROR_MESSAGE = ("{}:{} in target '{}':\n"
                  "  Source file '{}'\n"
                  "  crosses boundary of package '{}'.\n")
 
 
 class Logger(object):
   def __init__(self, messages_left=None):
     self.log_level = DISPLAY_LEVEL
     self.messages_left = messages_left
 
-  def log(self, build_file_path, line_number, target_name, source_file,
+  def Log(self, build_file_path, line_number, target_name, source_file,
           subpackage):
     if self.messages_left is not None:
       if not self.messages_left:
         self.log_level = IGNORE_LEVEL
       else:
         self.messages_left -= 1
     message = ERROR_MESSAGE.format(build_file_path, line_number, target_name,
                                    source_file, subpackage)
     logging.log(self.log_level, message)
 
@@ skipping 31 matching lines @@
     target_name = target_match.group('target_name')
     target_contents = target_match.group('target_contents')
     for sources_match in SOURCES_RE.finditer(target_contents):
       sources = sources_match.group('sources')
       for subpackages_match in subpackages_re.finditer(sources):
         subpackage = subpackages_match.group('subpackage')
         source_file = subpackages_match.group('source_file')
         line_number = subpackages_match.group('line_number')
         if subpackage:
           found_violations = True
-          logger.log(build_file_path, line_number, target_name, source_file,
+          logger.Log(build_file_path, line_number, target_name, source_file,
                      subpackage)
 
   return found_violations
 
 
 def CheckPackageBoundaries(root_dir, logger, build_files=None):
   packages = [root for root, _, files in os.walk(root_dir)
               if 'BUILD.gn' in files]
   default_build_files = [os.path.join(package, 'BUILD.gn')
                          for package in packages]
@@ skipping 23 matching lines @@
 
   logging.basicConfig(format=LOG_FORMAT)
   logging.getLogger().setLevel(DISPLAY_LEVEL)
   logger = Logger(args.max_messages)
 
   return CheckPackageBoundaries(args.root_dir, logger, args.build_files)
 
 
 if __name__ == '__main__':
   sys.exit(main())