[Android] Implement perf tests to platform mode.

build/android/test_runner.py

 #!/usr/bin/env python
 #
 # Copyright 2013 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 all types of tests from one unified interface."""
 
 import argparse
 import collections
@@ skipping 570 matching lines @@
       '-f', '--test-filter',
       help=('Test filter (will match against the names listed in --steps).'))
   group.add_argument(
       '--dry-run', action='store_true',
       help='Just print the steps without executing.')
   # Uses 0.1 degrees C because that's what Android does.
   group.add_argument(
       '--max-battery-temp', type=int,
       help='Only start tests when the battery is at or below the given '
            'temperature (0.1 C)')
   group.add_argument('single_step_command', nargs='*', action=SingleStepAction,

[jbudorick, 2016/06/07 15:29:28]

Can you drop these onto the next line to match the way most of the other
arguments in this file are formatted?

[rnephew (Reviews Here), 2016/06/09 22:06:39]

Done. Except the ones that fully fit on one line.

                      help='If --single-step is specified, the command to run.')
   group.add_argument('--min-battery-level', type=int,
                      help='Only starts tests when the battery is charged above '
                           'given level.')
   group.add_argument('--known-devices-file', help='Path to known device list.')
+  group.add_argument(
+      '--repeat', dest='repeat', type=int, default=0,
+      help='Number of times to repeat the specified set of tests.')
+  group.add_argument('--break-on-failure', '--break_on_failure',
+                     dest='break_on_failure', action='store_true',
+                     help='Whether to break on failure.')
+  group.add_argument('--write-buildbot-json', action='store_true',
+                     help='Whether to output buildbot json.')
   AddCommonOptions(parser)
   AddDeviceOptions(parser)
 
 
 def ProcessPerfTestOptions(args):
   """Processes all perf test options.
 
   Args:
     args: argparse.Namespace object.
 
@@ skipping 165 matching lines @@
           'Did not find device %s among attached device. Attached devices: %s'
           % (test_device, ', '.join(attached_devices)))
     return test_device
 
   else:
     if not attached_devices:
       raise device_errors.NoDevicesError()
     return sorted(attached_devices)
 
 
+_DEFAULT_PLATFORM_MODE_TESTS = [' gtest', 'instrumentation', 'perf']

[jbudorick, 2016/06/07 15:29:28]

Again, this CL shouldn't jump perf tests to default on.

[rnephew (Reviews Here), 2016/06/09 22:06:39]

Thats why I added the do not submit to the name, I do not intend to submit
these; I just haven't deleted it from the patchsets yet so I dont have to keep
adding and removing it for local runs (I know I could jus tuse the flags for
platform mode, but I.. dont)

[rnephew (Reviews Here), 2016/06/27 20:23:25]

Removed it, renaming to reflect that this is gone.

 def RunTestsCommand(args): # pylint: disable=too-many-return-statements
   """Checks test type and dispatches to the appropriate function.
 
   Args:
     args: argparse.Namespace object.
 
   Returns:
     Integer indicated exit code.
 
   Raises:
     Exception: Unknown command name passed in, or an exception from an
         individual test runner.
   """
   command = args.command
 
   ProcessCommonOptions(args)
   logging.info('command: %s', ' '.join(sys.argv))
-
-  if args.enable_platform_mode or command in ('gtest', 'instrumentation'):
+  if args.enable_platform_mode or command in _DEFAULT_PLATFORM_MODE_TESTS:
     return RunTestsInPlatformMode(args)
 
   forwarder.Forwarder.RemoveHostLog()
   if not ports.ResetTestServerPortAllocation():
     raise Exception('Failed to reset test server port.')
 
   def get_devices():
     return _GetAttachedDevices(args.blacklist_file, args.test_device,
                                args.enable_device_cache, args.num_retries)
 
   if command == 'linker':
     return _RunLinkerTests(args, get_devices())
   elif command == 'junit':
     return _RunJUnitTests(args)
   elif command == 'monkey':
     return _RunMonkeyTests(args, get_devices())
   elif command == 'perf':
     return _RunPerfTests(args, get_devices())
   elif command == 'python':
     return _RunPythonTests(args)
   else:
     raise Exception('Unknown test type.')
 
 
 _SUPPORTED_IN_PLATFORM_MODE = [
   # TODO(jbudorick): Add support for more test types.
   'gtest',
   'instrumentation',
+  'perf',
   'uirobot',
 ]
 
 
 def RunTestsInPlatformMode(args):
 
   def infra_error(message):
     logging.fatal(message)
     sys.exit(constants.INFRA_EXIT_CODE)
 
@@ skipping 138 matching lines @@
     if e.is_infra_error:
       return constants.INFRA_EXIT_CODE
     return constants.ERROR_EXIT_CODE
   except: # pylint: disable=W0702
     logging.exception('Unrecognized error occurred.')
     return constants.ERROR_EXIT_CODE
 
 
 if __name__ == '__main__':
   sys.exit(main())