web_dev_style: make CSS checker ignore /* <tags> in comments */

chrome/browser/web_dev_style/css_checker.py

 # Copyright 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.
 
 """Presubmit script for Chromium WebUI resources.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into depot_tools, and see
 http://www.chromium.org/developers/web-development-style-guide for the rules
 we're checking against here.
@@ skipping 367 matching lines @@
           'test': zero_width_lengths,
           'multiline': True,
         },
     ]
 
     results = []
     affected_files = self.input_api.AffectedFiles(include_deletes=False,
                                                   file_filter=self.file_filter)
     files = []
     for f in affected_files:
-      file_contents = '\n'.join(f.NewContents())
       path = f.LocalPath()
+
+      is_html = path.endswith('.html')
+      if not is_html and not path.endswith('.css'):
+        continue
+
+      # Remove all /*comments*/, @at-keywords, and grit <if|include> tags; we're
+      # not using a real parser. TODO(dbeam): Check alpha in <if> blocks.
+      file_contents = _remove_all('\n'.join(f.NewContents()))
+
       # Handle CSS files and HTML files with inline styles.
-      if path.endswith('.html'):
+      if is_html:
         file_contents = _extract_inline_style(file_contents)
 
-      if path.endswith('.html') or path.endswith('.css'):
-        # Remove all /*comments*/, @at-keywords, and grit <if|include> tags;
-        # we're not using a real parser. TODO(dbeam): Check alpha in <if>
-        # blocks.
-        file_contents = _remove_all(file_contents)
-        files.append((path, file_contents))
+      files.append((path, file_contents))
 
     for f in files:
       file_errors = []
       for check in added_or_modified_files_checks:
         # If the check is multiline, it receives the whole file and gives us
         # back a list of things wrong. If the check isn't multiline, we pass it
         # each line and the check returns something truthy if there's an issue.
         if ('multiline' in check and check['multiline']):
           assert not 'after' in check
           check_errors = check['test'](f[1])
@@ skipping 10 matching lines @@
                 error += check['after'](line)
               check_errors.append(error)
           if len(check_errors) > 0:
             file_errors.append('- %s\n%s' %
                 (check['desc'], '\n'.join(check_errors)))
       if file_errors:
         results.append(self.output_api.PresubmitPromptWarning(
             '%s:\n%s' % (f[0], '\n\n'.join(file_errors))))
 
     return results