web_dev_style: add presubmit for superfluous </include> tags.

chrome/browser/web_dev_style/resource_checker.py

Index: chrome/browser/web_dev_style/resource_checker.py
diff --git a/chrome/browser/web_dev_style/resource_checker.py b/chrome/browser/web_dev_style/resource_checker.py
new file mode 100644
index 0000000000000000000000000000000000000000..860d88c6c305b11d548c7f4a380ff812aeefb5d0
--- /dev/null
+++ b/chrome/browser/web_dev_style/resource_checker.py
@@ -0,0 +1,45 @@
+# Copyright 2014 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.
+
+"""
+Presubmit for Chromium HTML/CSS/JS resources. See chrome/browser/PRESUBMIT.py.
+"""
+
+import regex_check
+
+
+class ResourceChecker(object):
+  def __init__(self, input_api, output_api, file_filter=None):
+    self.input_api = input_api
+    self.output_api = output_api
+    self.file_filter = file_filter
+
+  def IncludeCheck(self, line_number, line):
+    return regex_check.RegexCheck(self.input_api.re, line_number, line,
+        "(</include>)", "</include> is unnecessary. Please remove.")
+
+  def RunChecks(self):
+    """Check for violations of the Chromium web development style guide. See
+       http://chromium.org/developers/web-development-style-guide
+    """
+    results = []
+
+    affected_files = self.input_api.change.AffectedFiles(
+        file_filter=self.file_filter, include_deletes=False)
+
+    for f in affected_files:
+      errors = []
+
+      for line_number, line in enumerate(f.NewContents(), start=1):
+        error = self.IncludeCheck(line_number, line)
+        if error:
+          errors.append(error)
+
+      if errors:
+        abs_local_path = f.AbsoluteLocalPath()
+        file_indicator = 'Found resources style issues in %s' % abs_local_path
+        prompt_msg = file_indicator + '\n\n' + '\n'.join(errors) + '\n'
+        results.append(self.output_api.PresubmitPromptWarning(prompt_msg))
+
+    return results