Exclude ChromeOS directories from Mac coverage lines....

build/mac/chrome_mac.croc

Index: build/mac/chrome_mac.croc
===================================================================
--- build/mac/chrome_mac.croc	(revision 59321)
+++ build/mac/chrome_mac.croc	(working copy)
@@ -6,11 +6,16 @@
   'rules' : [
     # Specify inclusions before exclusions, since rules are in order.
 
-    # Don't include linux or windows specific files
+    # Don't include chromeos, linux, or windows specific files
     {
-      'regexp' : '.*(_|/)(linux|win|views)(\\.|_)',
+      'regexp' : '.*(_|/)(chromeos|linux|win|views)(\\.|_)',
       'include' : 0,
     },
+    # Don't include ChromeOS dirs
+    {
+      'regexp' : '.*/chromeos/',

[bulach, 2010/10/04 09:42:05]

you may want to add this rule to chrome_linux.croc, or perhaps change the regexp
above to add trailing slash and remove this entirely:
.*(_|/)(chromeos|linux|win|views)(\\.|_|/)

[John Grabowski, 2010/10/04 20:21:39]

I would prefer to have 2 regexps which are marginally simpler.
E.g. one for filenames, one for directories.
regexps are hard enough to parse as it is.

Note chrome_linux.croc excludes both /chromeos/ and *_chromeos* with this CL. 
(But linux also ignores views directories, so the regexp doesn't match exactly.)

Thanks for the scrutiny.

+      'include' : 0,
+    },
 
     # Groups
     {