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

build/android/test_runner.py

Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index f4b6a4c9a3320db1e7d99253f92d8d8d6292843d..28525d57b721c870bc8edbd89081f25002b2d7b4 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -113,8 +113,23 @@ def AddCommonOptions(parser):
                      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: