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

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

Issue 2679393004: Fix all "dangerous-default-value" pylint warnings. (Closed)
Patch Set: Created 3 years, 10 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 b7938149ec5ad972b4c0e7dd996e5d77bb44098d..346f56db488638dc8595b068b01f2f178df027bc 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
@@ -206,7 +206,7 @@ class LayoutTestRunner(object):
def _handle_finished_test_list(self, worker_name, list_name):
pass
- def _handle_finished_test(self, worker_name, result, log_messages=[]):
+ def _handle_finished_test(self, worker_name, result, log_messages=None): # pylint: disable=unused-argument
self._update_summary_with_result(self._current_run_results, result)
def _handle_device_failed(self, worker_name, list_name, remaining_tests):

Powered by Google App Engine
This is Rietveld 408576698