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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py

Issue 2256793002: Make docstrings more consistent using format-webkitpy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make indentation of final quote based on parse tree (indentation prior to docstring node) rather th… Created 4 years, 3 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: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py
index c7213afb2ddadc3769e957bc8f3d48c93cad02e1..b46e9c22af2ea37967294316eeefc1732c1adde7 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py
@@ -239,7 +239,8 @@ class Worker(object):
def start(self):
"""This method is called when the object is starting to be used and it is safe
for the object to create state that does not need to be pickled (usually this means
- it is called in a child process)."""
+ it is called in a child process).
+ """
self._host = self._caller.host
self._filesystem = self._host.filesystem
self._port = self._host.port_factory.get(self._options.platform, self._options)
@@ -381,7 +382,8 @@ class Sharder(object):
def _shard_in_two(self, test_inputs):
"""Returns two lists of shards, one with all the tests requiring a lock and one with the rest.
- This is used when there's only one worker, to minimize the per-shard overhead."""
+ This is used when there's only one worker, to minimize the per-shard overhead.
+ """
locked_inputs = []
unlocked_inputs = []
for test_input in test_inputs:
@@ -402,7 +404,8 @@ class Sharder(object):
def _shard_every_file(self, test_inputs, run_singly):
"""Returns two lists of shards, each shard containing a single test file.
- This mode gets maximal parallelism at the cost of much higher flakiness."""
+ This mode gets maximal parallelism at the cost of much higher flakiness.
+ """
locked_shards = []
unlocked_shards = []
virtual_inputs = []
@@ -432,7 +435,8 @@ class Sharder(object):
"""Returns two lists of shards, each shard containing all the files in a directory.
This is the default mode, and gets as much parallelism as we can while
- minimizing flakiness caused by inter-test dependencies."""
+ minimizing flakiness caused by inter-test dependencies.
+ """
locked_shards = []
unlocked_shards = []
unlocked_slow_shards = []
@@ -473,7 +477,8 @@ class Sharder(object):
def _resize_shards(self, old_shards, max_new_shards, shard_name_prefix):
"""Takes a list of shards and redistributes the tests into no more
- than |max_new_shards| new shards."""
+ than |max_new_shards| new shards.
+ """
# This implementation assumes that each input shard only contains tests from a
# single directory, and that tests in each shard must remain together; as a

Powered by Google App Engine
This is Rietveld 408576698