cc: Simplify the presubmit regex used to identify paths and files.

cc/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 cc.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into depot_tools.
 """
 
 import re
 import string
 
-CC_SOURCE_FILES=(r'^cc/.*\.(cc|h)$', r'^cc\\.*\.(cc|h)$')
+CC_SOURCE_FILES=(r'^cc[\\\/].*\.(cc|h)$')

[tfarina, 2015/01/31 00:24:55]

Not sure if it is something wrong with this new way or something else.

Maybe you guys are more familiar with Regex will spot it.

[danakj, 2015/01/31 00:45:35]

From git history, this used to be

1d99317a7 (danakj@chromium.org        2012-10-18 18:15:04 +0000  14)
CC_SOURCE_FILES=(r'^cc/.*\.(cc|h)$',)

Note the , at the end.

 
 def CheckChangeLintsClean(input_api, output_api):
   input_api.cpplint._cpplint_state.ResetErrorCounts()  # reset global state
   source_filter = lambda x: input_api.FilterSourceFile(
     x, white_list=CC_SOURCE_FILES, black_list=None)
   files = [f.AbsoluteLocalPath() for f in
            input_api.AffectedSourceFiles(source_filter)]
   level = 1  # strict, but just warn
 
   for file_name in files:
@@ skipping 311 matching lines @@
   results += FindUselessIfdefs(input_api, output_api)
   results += input_api.canned_checks.CheckPatchFormatted(input_api, output_api)
   return results
 
 def GetPreferredTryMasters(project, change):
   return {
     'tryserver.blink': {
       'linux_blink_rel': set(['defaulttests']),
     },
   }