Android: adds device affinity for perf tests.

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 a0c3b72fe03e55b58ccfb099956b06fba90569e0..897a2bd1fad0fa7a0f08f69fcf13cc97de0e15fd 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -633,17 +633,21 @@ def _RunMonkeyTests(options, error_func, devices):
   return exit_code
 
 
-def _RunPerfTests(options, args, error_func, devices):
+def _RunPerfTests(options, args, error_func):
   """Subcommand of RunTestsCommands which runs perf tests."""
   perf_options = ProcessPerfTestOptions(options, args, error_func)
   # Just print the results from a single previously executed step.
   if perf_options.print_step:
     return perf_test_runner.PrintTestOutput(perf_options.print_step)
 
-  runner_factory, tests = perf_setup.Setup(perf_options)
+  runner_factory, tests, devices = perf_setup.Setup(perf_options)
 
+  # shard=False means that each device will get the full list of tests
+  # and then each one will decide their own affinity.
+  # shard=True means each device will pop the next test available from a queue,
+  # which increases throughput but have no affinity.
   results, _ = test_dispatcher.RunTests(

[jbudorick, 2014/06/03 13:52:52]

FTR, I am ok with the perf-specific affinity sharding.

[bulach, 2014/06/03 14:25:52]

thanks!
I'm very happy that all the sharding building blocks already support this mode,
and this change is only confined to Perf :)

-      tests, runner_factory, devices, shard=True, test_timeout=None,
+      tests, runner_factory, devices, shard=False, test_timeout=None,
       num_retries=options.num_retries)
 
   report_results.LogFull(
@@ -731,7 +735,7 @@ def RunTestsCommand(command, options, args, option_parser):
   elif command == 'monkey':
     return _RunMonkeyTests(options, option_parser.error, devices)
   elif command == 'perf':
-    return _RunPerfTests(options, args, option_parser.error, devices)
+    return _RunPerfTests(options, args, option_parser.error)
   else:
     raise Exception('Unknown test type.')