Revert of Add presubmit rule banning relative header includes

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 depot_tools.
 """
 
@@ skipping 2140 matching lines @@
     return []
 
   return [output_api.PresubmitPromptWarning("""
 Use of => operator detected in:
 %s
 Please ensure your code does not run on iOS9 (=> (arrow) does not work there).
 https://chromium.googlesource.com/chromium/src/+/master/docs/es6_chromium.md#Arrow-Functions
 """ % "\n".join("  %s:%d\n" % line for line in arrow_lines))]
 
 
-def _CheckForRelativeIncludes(input_api, output_api):
-  from cpp_checker import CppChecker
-
-  bad_files = {}
-  for f in input_api.AffectedFiles(include_deletes=False):
-    if (f.LocalPath().startswith('third_party') and
-      not f.LocalPath().startswith('third_party/WebKit') and
-      not f.LocalPath().startswith('third_party\\WebKit')):
-      continue
-
-    if not CppChecker.IsCppFile(f.LocalPath()):
-      continue
-
-    relative_includes = [line for line_num, line in f.ChangedContents()
-                         if "#include" in line and "../" in line]
-    bad_files[f.LocalPath()] = relative_includes
-
-  if not bad_files:
-    return []
-
-  error_descriptions = []
-  for file_path, bad_lines in bad_files.iteritems():
-    error_description = file_path
-    for line in bad_lines:
-      error_description += '\n    ' + line
-    error_descriptions.append(error_description)
-
-  results = []
-  results.append(output_api.PresubmitError(
-        'You added one or more relative #include paths (including "../").\n'
-        'These shouldn\'t be used because they can be used to include headers\n'
-        'from code that\'s not correctly specified as a dependency in the\n'
-        'relevant BUILD.gn file(s).',
-        error_descriptions))
-
-  return results
-
 def _AndroidSpecificOnUploadChecks(input_api, output_api):
   """Groups checks that target android code."""
   results = []
   results.extend(_CheckAndroidCrLogUsage(input_api, output_api))
   results.extend(_CheckAndroidNewMdpiAssetLocation(input_api, output_api))
   results.extend(_CheckAndroidToastUsage(input_api, output_api))
   results.extend(_CheckAndroidTestAnnotationUsage(input_api, output_api))
   return results
 
 
@@ skipping 41 matching lines @@
   results.extend(_CheckNoDeprecatedJs(input_api, output_api))
   results.extend(_CheckParseErrors(input_api, output_api))
   results.extend(_CheckForIPCRules(input_api, output_api))
   results.extend(_CheckForWindowsLineEndings(input_api, output_api))
   results.extend(_CheckSingletonInHeaders(input_api, output_api))
   results.extend(_CheckPydepsNeedsUpdating(input_api, output_api))
   results.extend(_CheckJavaStyle(input_api, output_api))
   results.extend(_CheckIpcOwners(input_api, output_api))
   results.extend(_CheckUselessForwardDeclarations(input_api, output_api))
   results.extend(_CheckForRiskyJsFeatures(input_api, output_api))
-  results.extend(_CheckForRelativeIncludes(input_api, output_api))
 
   if any('PRESUBMIT.py' == f.LocalPath() for f in input_api.AffectedFiles()):
     results.extend(input_api.canned_checks.RunUnitTestsInDirectory(
         input_api, output_api,
         input_api.PresubmitLocalPath(),
         whitelist=[r'^PRESUBMIT_test\.py$']))
   return results
 
 
 def _CheckPatchFiles(input_api, output_api):
@@ skipping 238 matching lines @@
       output_api,
       json_url='http://chromium-status.appspot.com/current?format=json'))
 
   results.extend(
       input_api.canned_checks.CheckPatchFormatted(input_api, output_api))
   results.extend(input_api.canned_checks.CheckChangeHasBugField(
       input_api, output_api))
   results.extend(input_api.canned_checks.CheckChangeHasDescription(
       input_api, output_api))
   return results