Run tests sorted by expected runtime.

Run tests sorted by expected runtime.

BUG=
R=jkummerow@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=21328

[Michael Achenbach, 2014-05-14 15:13:17]

PTAL

[Michael Achenbach, 2014-05-14 15:33:19]

Even test262 gets faster. Database size is ~2MB without and ~10MB with test262
per arch/mode. One TODO: optdebug is called debug within the runner and so is
the database. This might mix up values between debug and optdebug (even though
one could hope that the averages stay proportional).

[Jakob Kummerow, 2014-05-14 17:41:41]

LGTM with nits.

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/local/execu...
File tools/testrunner/local/execution.py (right):

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/local/execu...
tools/testrunner/local/execution.py:33: from . import perfdata
nit: alpha-sort please

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/local/execu...
tools/testrunner/local/execution.py:97: for test in sorted(self.tests,
key=lambda t: t.duration, reverse=True):
Hm... this is used by the network endpoint too, where in most cases the tests
list is already sorted as a result of how the work packets are assembled. If
that turns out to be a performance problem, we'll have to find a way to skip the
double sorting. But for now we can keep it, I think.

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
File tools/testrunner/network/network_execution.py (right):

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
tools/testrunner/network/network_execution.py:37: from ..local import execution
nit: alpha-sort please

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
tools/testrunner/network/network_execution.py:58: # But there is no super
constructor call. Check if this is a problem.
Shouldn't be a problem :-)
Unifying Runner and NetworkedRunner more is a bit tricky due to subtle interface
differences. Feel free to refactor them if you want; I tend to think that it's
not worth the effort.

[Michael Achenbach, 2014-05-15 07:16:36]

Done. Will investigate the stopability with Ctrl-C a bit more before landing...

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
File tools/testrunner/network/network_execution.py (right):

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
tools/testrunner/network/network_execution.py:37: from ..local import execution
On 2014/05/14 17:41:42, Jakob wrote:
> nit: alpha-sort please

Was moving the file in the end... didn't see the order problem.

https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/net...
tools/testrunner/network/network_execution.py:58: # But there is no super
constructor call. Check if this is a problem.
On 2014/05/14 17:41:42, Jakob wrote:
> Shouldn't be a problem :-)
> Unifying Runner and NetworkedRunner more is a bit tricky due to subtle
interface
> differences. Feel free to refactor them if you want; I tend to think that it's
> not worth the effort.

This TODO is just a remainder for when I might do this refactoring (in a future
CL).

[Michael Achenbach, 2014-05-15 12:02:25]

Committed patchset #3 manually as r21328 (presubmit successful).