Use inclusive parameter names in PRESUBMIT.py Bug: 1098562 Change-Id: I10e83b99a480d2d96b911e26b09e8db5d34a082e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333301 Commit-Queue: Josip Sokcevic <[email protected]> Reviewed-by: Scott Violet <[email protected]> Reviewed-by: Dirk Pranke <[email protected]> Reviewed-by: Peter Kasting <[email protected]> Cr-Commit-Position: refs/heads/master@{#795088}
diff --git a/PRESUBMIT_test_mocks.py b/PRESUBMIT_test_mocks.py index f3e928c..0a9e5a5 100644 --- a/PRESUBMIT_test_mocks.py +++ b/PRESUBMIT_test_mocks.py
@@ -61,8 +61,6 @@ """ DEFAULT_FILES_TO_SKIP = () - # TODO(https://crbug.com/1098562): Remove once no longer used) - DEFAULT_BLACK_LIST = () def __init__(self): self.canned_checks = MockCannedChecks() @@ -95,11 +93,7 @@ return self.AffectedFiles(file_filter=file_filter) def FilterSourceFile(self, file, - files_to_check=(), files_to_skip=(), - # TODO(https://crbug.com/1098562): Remove once no longer used - white_list=(), black_list=()): - files_to_check = files_to_check or white_list - files_to_skip = files_to_skip or black_list + files_to_check=(), files_to_skip=()): local_path = file.LocalPath() found_in_files_to_check = not files_to_check if files_to_check: