Use non-copying sorting in test harness.

tools/testrunner/local/execution.py

Index: tools/testrunner/local/execution.py
diff --git a/tools/testrunner/local/execution.py b/tools/testrunner/local/execution.py
index 80a5989be2f255a672b872adca74b284ad2ef49c..9c8a72d0236ae736f2ae717d7dcf9f7705bca1cd 100644
--- a/tools/testrunner/local/execution.py
+++ b/tools/testrunner/local/execution.py
@@ -66,6 +66,8 @@ class Runner(object):
     self.tests = [ t for s in suites for t in s.tests ]
     for t in self.tests:

[Michael Achenbach, 2014/05/28 12:37:28]

I also meant that "if not self.context.no_sorting:" could include lines 67 and
68. No need to fetch perf data and initialize it if we don't sort based on it.

[Michael Starzinger, 2014/05/28 12:51:07]

Done.

       t.duration = self.perfdata.FetchPerfData(t) or 1.0
+    if not self.context.no_sorting:
+      self.tests.sort(key=lambda t: t.duration, reverse=True)
     self._CommonInit(len(self.tests), progress_indicator, context)
 
   def _CommonInit(self, num_tests, progress_indicator, context):
@@ -94,8 +96,6 @@ class Runner(object):
     # while the queue is filled.
     queue = []
     queued_exception = None
-    if not self.context.no_sorting:
-      self.tests = sorted(self.tests, key=lambda t: t.duration, reverse=True)
     for test in self.tests:
       assert test.id >= 0
       test_map[test.id] = test