Make the IsClangWinBuild function GN friendly

scripts/common/chromium_utils.py

Index: scripts/common/chromium_utils.py
diff --git a/scripts/common/chromium_utils.py b/scripts/common/chromium_utils.py
index 8dd27ef8adf9ea913501b72ad6583b6c2da2d10b..69c442000a592cb088f09c0ef4d5b3a62a07f394 100644
--- a/scripts/common/chromium_utils.py
+++ b/scripts/common/chromium_utils.py
@@ -2005,24 +2005,19 @@ def GetSlaveNamesForBuilder(builders, builder_name):
   return slaves
 
 def IsClangWinBuild(build_dir, target):
-  """Check if a ninja build has been build with Clang on Windows.
-
-  This checks the build.ninja file from the build directory to see which
-  compiler has been used for a build.
-  """
-  # TODO(sebmarchand): Make this works for gn builds, currently only the gyp one
-  #     is supported.
+  """Check if a ninja build has been build with Clang on Windows."""
   if not IsWindows():
     return False
-  build_file = os.path.join(build_dir, target, 'build.ninja')
-  if not os.path.isfile(build_file):
+
+  gn_file = os.path.join(build_dir, target, 'args.gn')
+  if not os.path.isfile(gn_file):
+    print 'WARNING:  Unable to find the args.gn file.'
     return False
-  # Matches e.g. "cl_x86 = path/to/clang-cl.exe"
-  clang_cl_re = re.compile(
-      r'^cl_x\d\d\s+\=\s+(?P<compiler_path>[^ ]+)\s.*$',
-      re.VERBOSE)
-  for line in open(build_file):
-    m = clang_cl_re.match(line)
-    if m:
-      return 'clang' in m.group('compiler_path')
+  # Matches e.g. "gn_arg = value"
+  gn_arg_re = re.compile(r'^(?P<flag>[^= ]+)\s*=\s*(?P<value>[^ \n]+)$',
+                         re.VERBOSE)

[Nico, 2016/08/25 15:14:15]

i don't think you're using any of re.VERBOSE's features in your regex --
consider removing this?

[Sébastien Marchand, 2016/08/25 15:16:08]

Done.

+  for line in open(gn_file):
+    m = gn_arg_re.match(line)
+    if m and m.group('flag') == 'is_clang':
+      return m.group('value') == 'true'
   return False