Refactor android test results logging.

build/android/run_monkey_test.py

 #!/usr/bin/env python
 # 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.
 
 """Runs the Monkey tests on one or more devices."""
 import logging
 import optparse
 import random
 import sys
@@ skipping 90 matching lines @@
   if not attached_devices:
     raise Exception('You have no devices attached or visible!')
 
   # Actually run the tests.
   logging.debug('Running monkey tests.')
   available_tests *= len(attached_devices)
   options.ensure_value('shard_retries', 1)
   sharder = python_test_sharder.PythonTestSharder(
       attached_devices, available_tests, options)
   result = sharder.RunShardedTests()
-  result.LogFull('Monkey', 'Monkey', options.build_type, available_tests)
+  result.LogFull('Monkey',

[Isaac (away), 2012/12/18 04:17:43]

named arguments here too?

[frankf, 2012/12/18 21:33:50]

Done.

+                 'Monkey',
+                 build_type=options.build_type)
   result.PrintAnnotation()
 
 
 def main():
   desc = 'Run the Monkey tests on 1 or more devices.'
   parser = optparse.OptionParser(description=desc)
   test_options_parser.AddBuildTypeOption(parser)
   parser.add_option('--package-name', help='Allowed package.')
   parser.add_option('--activity-name',
                     default='com.google.android.apps.chrome.Main',
@@ skipping 24 matching lines @@
     parser.error('Missing package name')
 
   if options.category:
     options.category = options.category.split(',')
 
   DispatchPythonTests(options)
 
 
 if __name__ == '__main__':
   main()