[chrome_remote_control] Add pylint to PRESUMMIT and fix lint

tools/chrome_remote_control/chrome_remote_control/desktop_browser_finder_unittest.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 os as real_os
 import unittest
 
 from chrome_remote_control import browser_options
 from chrome_remote_control import desktop_browser_finder
-from chrome_remote_control.system_stub import *
+from chrome_remote_control.system_stub import (
+  OSModuleStub,
+  SysModuleStub
+  )
 
 # This file verifies the logic for finding a browser instance on all platforms
 # at once. It does so by providing stubs for the OS/sys/subprocess primitives
 # that the underlying finding logic usually uses to locate a suitable browser.
 # We prefer this approach to having to run the same test on every platform on
 # which we want this code to work.
 
 class StubSubprocess(object):
   def __init__(self):
     self.call_hook = None
@@ skipping 55 matching lines @@
   def setUp(self):
     super(LinuxFindTest, self).setUp()
 
     self._sys_stub.platform = 'linux2'
     self._files.append('/foo/chrome')
     self._files.append('../../../out/Release/chrome')
     self._files.append('../../../out/Debug/chrome')
     self._files.append('../../../out/Release/content_shell')
     self._files.append('../../../out/Debug/content_shell')
 
-    self._has_google_chrome_on_path = False
+    self.has_google_chrome_on_path = False
     this = self
-    def call_hook(*args, **kwargs):
-      if this._has_google_chrome_on_path:
+    def call_hook(*args, **kwargs): # pylint: disable=W0613
+      if this.has_google_chrome_on_path:
         return 0
       raise OSError('Not found')
     self._subprocess_stub.call_hook = call_hook
 
   def testFindAllWithExact(self):
     types = self.DoFindAllTypes()
     self.assertEquals(
         set(types),
         set(['debug', 'release',
              'content-shell-debug', 'content-shell-release']))
 
   def testFindWithProvidedExecutable(self):
     self._options.browser_executable = '/foo/chrome'
     self.assertTrue('exact' in self.DoFindAllTypes())
 
   def testFindUsingDefaults(self):
-    self._has_google_chrome_on_path = True
+    self.has_google_chrome_on_path = True
     self.assertTrue('release' in self.DoFindAllTypes())
 
     del self._files[1]
-    self._has_google_chrome_on_path = True
+    self.has_google_chrome_on_path = True
     self.assertTrue('system' in self.DoFindAllTypes())
 
-    self._has_google_chrome_on_path = False
+    self.has_google_chrome_on_path = False
     del self._files[1]
     self.assertEquals(['content-shell-debug', 'content-shell-release'],
                       self.DoFindAllTypes())
 
   def testFindUsingRelease(self):
     self.assertTrue('release' in self.DoFindAllTypes())
 
 
 class WinFindTest(FindTestBase):
   def setUp(self):
@@ skipping 20 matching lines @@
 
   def testFindAllWithExact(self):
     self._options.browser_executable = 'c:\\tmp\\chrome.exe'
     types = self.DoFindAllTypes()
     self.assertEquals(
         set(types),
         set(['exact',
              'debug', 'release',
              'content-shell-debug', 'content-shell-release',
              'system', 'canary']))