PRESUBMIT doesn't need to check for old callback system

PRESUBMIT.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Top-level presubmit script for Chromium.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into gcl.
 """
 
@@ skipping 164 matching lines @@
       if 'FRIEND_TEST(' in line:
         problems.append('    %s:%d' % (f.LocalPath(), line_num))
 
   if not problems:
     return []
   return [output_api.PresubmitPromptWarning('Chromium code should not use '
       'gtest\'s FRIEND_TEST() macro. Include base/gtest_prod_util.h and use '
       'FRIEND_TEST_ALL_PREFIXES() instead.\n' + '\n'.join(problems))]
 
 
-def _CheckNoNewOldCallback(input_api, output_api):
-  """Checks to make sure we don't introduce new uses of old callbacks."""
-
-  def HasOldCallbackKeywords(line):
-    """Returns True if a line of text contains keywords that indicate the use
-    of the old callback system.
-    """
-    return ('NewRunnableMethod' in line or
-            'NewCallback' in line or
-            input_api.re.search(r'\bCallback\d<', line) or
-            input_api.re.search(r'\bpublic Task\b', line) or
-            'public CancelableTask' in line)
-
-  problems = []
-  file_filter = lambda f: f.LocalPath().endswith(('.cc', '.h'))
-  for f in input_api.AffectedFiles(file_filter=file_filter):
-    if not any(HasOldCallbackKeywords(line) for line in f.NewContents()):
-      continue
-    for line_num, line in f.ChangedContents():
-      if HasOldCallbackKeywords(line):
-        problems.append('    %s:%d' % (f.LocalPath(), line_num))
-
-  if not problems:
-    return []
-  return [output_api.PresubmitPromptWarning('The old callback system is '
-      'deprecated. If possible, use base::Bind and base::Callback instead.\n' +
-      '\n'.join(problems))]
-
-
 def _CommonChecks(input_api, output_api):
   """Checks common to both upload and commit."""
   results = []
   results.extend(input_api.canned_checks.PanProjectChecks(
       input_api, output_api, excluded_paths=_EXCLUDED_PATHS))
   results.extend(_CheckNoInterfacesInBase(input_api, output_api))
   results.extend(_CheckAuthorizedAuthor(input_api, output_api))
   results.extend(
     _CheckNoProductionCodeUsingTestOnlyFunctions(input_api, output_api))
   results.extend(_CheckNoIOStreamInHeaders(input_api, output_api))
   results.extend(_CheckNoNewWStrings(input_api, output_api))
   results.extend(_CheckNoDEPSGIT(input_api, output_api))
   results.extend(_CheckNoFRIEND_TEST(input_api, output_api))
-  results.extend(_CheckNoNewOldCallback(input_api, output_api))
   return results
 
 
 def _CheckSubversionConfig(input_api, output_api):
   """Verifies the subversion config file is correctly setup.
 
   Checks that autoprops are enabled, returns an error otherwise.
   """
   join = input_api.os_path.join
   if input_api.platform == 'win32':
@@ skipping 113 matching lines @@
   # 2 trybots (compared against ~45 for Mac and Linux).
   # If any file matches something compiled on the main waterfall
   # android builder, use the android try server.
   android_re_list = ('^base/', '^ipc/', '^net/', '^sql/', '^jingle/',
                      '^build/common.gypi$')
   for f in affected_files:
     if any(re.search(r, f) for r in android_re_list):
       preferred.append('android')
       break
   return preferred