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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.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
« no previous file with comments | « no previous file | third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py
index 63a84c786a7bd8ec7d1dd7c64bc9a2a0af6d3ffa..461d27d9da7d9b6efb8744352177ad6586ab5371 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py
@@ -79,7 +79,7 @@ class FileSystem(object):
def exists(self, path):
return os.path.exists(path)
- def files_under(self, path, dirs_to_skip=[], file_filter=None):
+ def files_under(self, path, dirs_to_skip=None, file_filter=None):
"""Return the list of all files under the given path in topdown order.
Args:
@@ -90,6 +90,8 @@ class FileSystem(object):
each file found. The file is included in the result if the
callback returns True.
"""
+ dirs_to_skip = dirs_to_skip or []
+
def filter_all(fs, dirpath, basename):
return True
« no previous file with comments | « no previous file | third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698