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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_input.py

Issue 2329263002: Run format-webkitpy and fix long lines. (Closed)
Patch Set: 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/models/test_input.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_input.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_input.py
index e6f3f472de8d83fff6c9cce3df59b1909593cb0a..77d5852945d6346735277c9c38a51890c4826d91 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_input.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_input.py
@@ -34,15 +34,24 @@ class TestInput(object):
def __init__(self, test_name, timeout=None, requires_lock=None, reference_files=None,
should_run_pixel_test=None, should_add_missing_baselines=True):
# TestInput objects are normally constructed by the manager and passed
- # to the workers, but these some fields are set lazily in the workers where possible
- # because they require us to look at the filesystem and we want to be able to do that in parallel.
+ # to the workers, but these some fields are set lazily in the workers
+ # where possible, because they require us to look at the filesystem,
+ # and we want to be able to do that in parallel.
self.test_name = test_name
- self.timeout = timeout # in msecs; should rename this for consistency
+ self.timeout = timeout # in msecs; FIXME: should rename this for consistency
self.requires_lock = requires_lock
self.reference_files = reference_files
self.should_run_pixel_test = should_run_pixel_test
self.should_add_missing_baselines = should_add_missing_baselines
def __repr__(self):
- return "TestInput('%s', timeout=%s, requires_lock=%s, reference_files=%s, should_run_pixel_test=%s, should_add_missing_baselines=%s)" % (
- self.test_name, self.timeout, self.requires_lock, self.reference_files, self.should_run_pixel_test, self.should_add_missing_baselines)
+ return (
+ "TestInput('%s', timeout=%s, requires_lock=%s, "
+ "reference_files=%s, should_run_pixel_test=%s, "
+ "should_add_missing_baselines=%s)" % (
+ self.test_name,
+ self.timeout,
+ self.requires_lock,
+ self.reference_files,
+ self.should_run_pixel_test,
+ self.should_add_missing_baselines))

Powered by Google App Engine
This is Rietveld 408576698