Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1243)

Unified Diff: Tools/Scripts/webkitpy/layout_tests/port/driver.py

Issue 1161863003: Add an additional content_shell per worker for running virtual tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@virtual_reference_flags
Patch Set: fix docstring for Driver.run_test() Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Tools/Scripts/webkitpy/layout_tests/port/driver.py
diff --git a/Tools/Scripts/webkitpy/layout_tests/port/driver.py b/Tools/Scripts/webkitpy/layout_tests/port/driver.py
index 832056a8b3cfc3521aa3871004da1f0977da2c14..b8199ae030c949e01b9d2e650630013e9ab47a54 100644
--- a/Tools/Scripts/webkitpy/layout_tests/port/driver.py
+++ b/Tools/Scripts/webkitpy/layout_tests/port/driver.py
@@ -143,9 +143,10 @@ class Driver(object):
def run_test(self, driver_input, stop_when_done):
"""Run a single test and return the results.
- Note that it is okay if a test times out or crashes and leaves
- the driver in an indeterminate state. The upper layers of the program
- are responsible for cleaning up and ensuring things are okay.
+ Note that it is okay if a test times out or crashes. content_shell
+ will be stopped when the test ends, and then restarted for the next
+ test when this function is invoked again. As part of the restart, the
+ state of Driver will be reset.
Returns a DriverOutput object.
"""

Powered by Google App Engine
This is Rietveld 408576698