test_runner.py Add --fast-local-dev flag

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 95 matching lines @@
                      help=('Specify the absolute path of the adb binary that '
                            'should be used.'))
   group.add_argument('--json-results-file', '--test-launcher-summary-output',
                      dest='json_results_file',
                      help='If set, will dump results in JSON form '
                           'to specified file.')
   group.add_argument('--logcat-output-dir',
                      help='If set, will dump logcats recorded during test run '
                           'to directory. File names will be the device ids.')
 
+  class FastLocalDevAction(argparse.Action):

[jbudorick, 2016/02/04 05:34:24]

I would normally be against a local class, but this is so specific that ...
maybe?

I also have no idea what the appropriate naming style is for it. We've done
lower_case_with_underscores for local functions to distinguish them as local
functions, but I feel like that'd be weird here, so I guess this is ok.

[agrieve, 2016/02/04 16:09:14]

Yeah, there's another one already on line 607, so at least it's consistent
there. 

+    def __call__(self, parser, namespace, values, option_string=None):
+      namespace.verbose_count = max(namespace.verbose_count, 1)
+      namespace.num_retries = 0
+      namespace.enable_device_cache = True
+      namespace.skip_clear_data = True
+      namespace.extract_test_list_from_filter = True
+
+  group.add_argument('--fast-local-dev', type=bool, nargs=0,
+                     action=FastLocalDevAction,
+                     help='Alias for: --verbose --num-retries=0 '
+                          '--enable-device-cache --skip-clear-data '
+                          '--extract-test-list-from-filter')
+
 def ProcessCommonOptions(args):
   """Processes and handles all common options."""
+
   run_tests_helper.SetLogLevel(args.verbose_count)
   constants.SetBuildType(args.build_type)
   if args.build_directory:
     constants.SetBuildDirectory(args.build_directory)
   if args.output_directory:
     constants.SetOutputDirectory(args.output_directory)
 
   devil_custom_deps = None
   if args.adb_path:
     devil_custom_deps = {
@@ skipping 978 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())