Fix python scripts in src/tools/

tools/code_coverage/coverage.py

 #!/bin/env python
-# Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+# Copyright (c) 2011 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.
 
 
 """Module to setup and generate code coverage data
 
 This module first sets up the environment for code coverage, instruments the
 binaries, runs the tests and collects the code coverage data.
 
 
@@ skipping 218 matching lines @@
 
     # Stop the counters
     counters_command = ('%s -shutdown' %
                         (os.path.join(self.tools_path, 'vsperfcmd.exe')))
     (retcode, output) = proc.RunCommandFull(counters_command,
                                             collect_output=True)
     logging.info('Counters shut down: %s' % (output))
     # Return the intermediate .coverage file
     return coverage_file
 
 

[Alexander Potapenko, 2011/11/28 10:29:35]

Please use 1 line between the class-level definitions:
http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Blank_Lines

   def Upload(self, list_coverage, upload_path, sym_path=None, src_root=None):
     """Upload the results to the dashboard.
 
     This method uploads the coverage data to a dashboard where it will be
     processed. On Windows, this method will first convert the .coverage file to
     the lcov format. This method needs to be called before the TearDown method.
 
     Args:
       list_coverage: The list of coverage data files to consoliate and upload.
       upload_path: Destination where the coverage data will be processed.
@@ skipping 109 matching lines @@
 
     cov.Upload(list_coverage,
                options.upload_path,
                os.path.join(options.src_root, 'chrome', 'Release'),
                options.src_root)
     cov.TearDown()
 
 
 if __name__ == '__main__':
   sys.exit(main())