[chrome_remote_control] Add pylint to PRESUMMIT and fix lint

tools/chrome_remote_control/chrome_remote_control/run_tests.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.
 import fnmatch
-import functools
 import logging
 import os
 import traceback
 import unittest
 
 from chrome_remote_control import browser_options
 
 def RequiresBrowserOfType(*types):
   def wrap(func):
     func._requires_browser_types = types
     return func
   return wrap
 
 def Discover(start_dir, pattern = 'test*.py', top_level_dir = None):
   if hasattr(unittest.defaultTestLoader, 'discover'):
-    return unittest.defaultTestLoader.discover(start_dir,
-                                               pattern,
-                                               top_level_dir)
+    return unittest.defaultTestLoader.discover( # pylint: disable=E1101
+      start_dir,
+      pattern,
+      top_level_dir)
 
   modules = []
   for dirpath, _, filenames in os.walk(start_dir):
     for filename in filenames:
       if not filename.endswith('.py'):
         continue
 
       if not fnmatch.fnmatch(filename, pattern):
         continue
 
       if filename.startswith('.') or filename.startswith('_'):
         continue
       name, _ = os.path.splitext(filename)
 
       relpath = os.path.relpath(dirpath, top_level_dir)
       fqn = relpath.replace('/', '.') + '.' + name
 
       # load the module
       try:
         module = __import__(fqn, fromlist=[True])
-      except:
+      except Exception:
         print 'While importing [%s]\n' % fqn
         traceback.print_exc()
         continue
       modules.append(module)
 
   loader = unittest.defaultTestLoader
   subsuites = []
   for module in modules:
     if hasattr(module, 'suite'):
       new_suite = module.suite()
@@ skipping 26 matching lines @@
   def IsTestSelected(test):
     if len(args) != 0:
       found = False
       for name in args:
         if name in test.id():
           found = True
       if not found:
         return False
 
     if hasattr(test, '_testMethodName'):
-      method = getattr(test, test._testMethodName)
+      method = getattr(test, test._testMethodName) # pylint: disable=W0212
       if hasattr(method, '_requires_browser_types'):
-        types = method._requires_browser_types
+        types = method._requires_browser_types # pylint: disable=W0212
         if browser_options.browser_type_for_unittests not in types:
           logging.debug('Skipping test %s because it requires %s' %
                         (test.id(), types))
           return False
 
     return True
 
   filtered_suite = FilterSuite(suite, IsTestSelected)
   runner = unittest.TextTestRunner(verbosity = 2)
   test_result = runner.run(filtered_suite)
@@ skipping 18 matching lines @@
                   default_options.browser_type)
     logging.error('Re-run with --browser=list to see available browser types.')
     return 1
 
   browser_options.options_for_unittests = default_options
   browser_options.browser_type_for_unittests = browser_to_create.browser_type
   olddir = os.getcwd()
   num_errors = 0
   try:
     os.chdir(top_level_dir)
-    for _ in range(default_options.run_test_repeat_count):
+    for _ in range(
+        default_options.run_test_repeat_count): # pylint: disable=E1101
       num_errors += DiscoverAndRunTests(start_dir, args, top_level_dir)
   finally:
     os.chdir(olddir)
     browser_options.options_for_unittests = None
 
   return num_errors