PRESUBMIT.py: update arg name to avoid warnings on newer depot_tools

Newer depot_tools have switched the argument naming convention again,
and cause presubmits to print warnings even when using
allow_list/block_list:
  presubmit_support.py:744: UserWarning: Use files_to_check in FilterSourceFile

The logic is still backwards compatible (the allow_list is used if the
new param is empty), but the warning is undesirable, as it is pure
noise.

Afaiu, as-is, this patch will force people to upgrade to a depot_tools
that understands the new parameter name. Which feels like an annoyance
for such a small change. One alternative would be to drop the argname
qualification, and use purely positional args for now (then it'll work
with either depot_tools version).

Change-Id: I2302928ffd7863b2429e5f33e55ffc07134d7101

PRESUBMIT.py

diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 65ee8a9..9ccc4b6 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -22,8 +22,8 @@
   def long_line_sources(x):
     return input.FilterSourceFile(
         x,
-        allow_list=".*",
-        block_list=[
+        files_to_check='.*',
+        files_to_skip=[
             'Android[.]bp', '.*[.]json$', '.*[.]sql$', '.*[.]out$',
             'test/trace_processor/.*/index$', '.*\bBUILD$', 'WORKSPACE',
             '.*/Makefile$', '/perfetto_build_flags.h$'
@@ -63,7 +63,7 @@
   # If no GN files were modified, bail out.
   def build_file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=('.*BUILD[.]gn$', '.*[.]gni$', 'BUILD\.extras', tool))
+        x, files_to_check=('.*BUILD[.]gn$', '.*[.]gni$', 'BUILD\.extras', tool))
 
   if not input_api.AffectedSourceFiles(build_file_filter):
     return []
@@ -81,7 +81,7 @@
   # If no GN files were modified, bail out.
   def build_file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=('.*BUILD[.]gn$', '.*[.]gni$', tool))
+        x, files_to_check=('.*BUILD[.]gn$', '.*[.]gni$', tool))
 
   if not input_api.AffectedSourceFiles(build_file_filter):
     return []
@@ -98,7 +98,7 @@
 
   def file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=['.*[.]cc$', '.*[.]h$', tool])
+        x, files_to_check=['.*[.]cc$', '.*[.]h$', tool])
 
   if not input_api.AffectedSourceFiles(file_filter):
     return []
@@ -134,7 +134,7 @@
   ]
 
   def file_filter(x):
-    return input_api.FilterSourceFile(x, allow_list=[r'.*\.h$', r'.*\.cc$'])
+    return input_api.FilterSourceFile(x, files_to_check=[r'.*\.h$', r'.*\.cc$'])
 
   errors = []
   for f in input_api.AffectedSourceFiles(file_filter):
@@ -151,7 +151,8 @@
   tool = 'tools/check_include_violations'
 
   def file_filter(x):
-    return input_api.FilterSourceFile(x, allow_list=['include/.*[.]h$', tool])
+    return input_api.FilterSourceFile(
+        x, files_to_check=['include/.*[.]h$', tool])
 
   if not input_api.AffectedSourceFiles(file_filter):
     return []
@@ -165,7 +166,7 @@
 
   def file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=['protos/perfetto/.*[.]proto$', '.*[.]h', tool])
+        x, files_to_check=['protos/perfetto/.*[.]proto$', '.*[.]h', tool])
 
   if not input_api.AffectedSourceFiles(file_filter):
     return []
@@ -182,7 +183,7 @@
 
   def build_file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=['protos/perfetto/.*[.]proto$', tool])
+        x, files_to_check=['protos/perfetto/.*[.]proto$', tool])
 
   if not input_api.AffectedSourceFiles(build_file_filter):
     return []
@@ -216,7 +217,7 @@
 
   def file_filter(x):
     return input_api.FilterSourceFile(
-        x, allow_list=['protos/perfetto/.*[.]proto$', tool])
+        x, files_to_check=['protos/perfetto/.*[.]proto$', tool])
 
   if not input_api.AffectedSourceFiles(file_filter):
     return []