Use non-copying sorting in test harness.

tools/testrunner/local/execution.py

 # Copyright 2012 the V8 project authors. All rights reserved.
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 #       notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 #       copyright notice, this list of conditions and the following
 #       disclaimer in the documentation and/or other materials provided
@@ skipping 77 matching lines @@
 
   def _RunInternal(self, jobs):
     pool = Pool(jobs)
     test_map = {}
     # TODO(machenbach): Instead of filling the queue completely before
     # pool.imap_unordered, make this a generator that already starts testing
     # 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)
+      self.tests.sort(key=lambda t: t.duration, reverse=True)

[Michael Achenbach, 2014/05/28 12:03:21]

Maybe move to line 68 right after the duration initialization and also skip that
initialization dependent on the flag.

[Michael Starzinger, 2014/05/28 12:13:30]

Done.

     for test in self.tests:
       assert test.id >= 0
       test_map[test.id] = test
       try:
         command = self.GetCommand(test)
       except Exception, e:
         # If this failed, save the exception and re-raise it later (after
         # all other tests have had a chance to run).
         queued_exception = e
         continue
@@ skipping 50 matching lines @@
            test.suite.GetFlagsForTestCase(test, self.context) +
            self.context.extra_flags)
     return cmd
 
 
 class BreakNowException(Exception):
   def __init__(self, value):
     self.value = value
   def __str__(self):
     return repr(self.value)