Done. Will investigate the stopability with Ctrl-C a bit more before
landing...
https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/network_execution.py
File tools/testrunner/network/network_execution.py (right):
https://codereview.chromium.org/273613005/diff/2/tools/testrunner/network/network_execution.py#newcode37
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/network_execution.py#newcode58
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).
https://codereview.chromium.org/273613005/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
