[chrome_remote_control] Add pylint to PRESUMMIT and fix lint

tools/chrome_remote_control/chrome_remote_control/browser_options.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 optparse
 import os
 import sys
 import shlex
 import logging
 
 from chrome_remote_control import browser_finder
 
 class BrowserOptions(optparse.Values):
   """Options to be used for discovering and launching a browser."""
 
   def __init__(self, browser_type=None):
     optparse.Values.__init__(self)
 
     self.browser_type = browser_type
     self.browser_executable = None
     self.chrome_root = None
     self.android_device = None
 
     self.dont_override_profile = False
     self.extra_browser_args = []
     self.show_stdout = False
 
+    self.cros_remote = None
+
     self.verbosity = 0
 
   def Copy(self):
     other = BrowserOptions()
     other.__dict__.update(self.__dict__)
     return other
 
   def CreateParser(self, *args, **kwargs):
     parser = optparse.OptionParser(*args, **kwargs)
 
@@ skipping 41 matching lines @@
     group = optparse.OptionGroup(parser, 'When things go wrong')
     group.add_option(
       '-v', '--verbose', action='count', dest='verbosity',
       help='Increase verbosity level (repeat as needed)')
     parser.add_option_group(group)
 
     real_parse = parser.parse_args
     def ParseArgs(args=None):
       defaults = parser.get_default_values()
       for k, v in defaults.__dict__.items():
-        if k in self.__dict__:
+        if k in self.__dict__ and self.__dict__[k] != None:
           continue
         self.__dict__[k] = v
-      ret = real_parse(args, self)
+      ret = real_parse(args, self) # pylint: disable=E1121
 
       if self.verbosity >= 2:
         logging.basicConfig(level=logging.DEBUG)
       elif self.verbosity:
         logging.basicConfig(level=logging.INFO)
       else:
         logging.basicConfig(level=logging.WARNING)
 
       if self.browser_executable and not self.browser_type:
         self.browser_type = 'exact'
       if not self.browser_executable and not self.browser_type:
         sys.stderr.write('Must provide --browser=<type>. ' +
                          'Use --browser=list for valid options.\n')
         sys.exit(1)
       if self.browser_type == 'list':
         types = browser_finder.GetAllAvailableBrowserTypes(self)
         sys.stderr.write('Available browsers:\n')
         sys.stdout.write('  %s\n' % '\n  '.join(types))
         sys.exit(1)
-      if self.extra_browser_args_as_string:
-        tmp = shlex.split(self.extra_browser_args_as_string)
+      if self.extra_browser_args_as_string: # pylint: disable=E1101
+        tmp = shlex.split(
+          self.extra_browser_args_as_string) # pylint: disable=E1101
         self.extra_browser_args.extend(tmp)
         delattr(self, 'extra_browser_args_as_string')
       return ret
     parser.parse_args = ParseArgs
     return parser
 
-"""
-This global variable can be set to a BrowserOptions object by the test harness
-to allow multiple unit tests to use a specific browser, in face of multiple
-options.
-"""
+# This global variable can be set to a BrowserOptions object by the test harness
+# to allow multiple unit tests to use a specific browser, in face of multiple
+# options.
 options_for_unittests = None