ninja windows: add flag support for code analysis (aka PREfast)

ninja windows: add flag support for code analysis (aka PREfast)

Also add test for same for both MSVS and ninja. This test is
only enabled on VS2012 for now. Theoretically, it could be
be enabled for VS2010 Premium too, but few people have that
installed, and we don't currently have a way to gate on that,
so just turn it on for 2012 (non-Express) where it's always
available.
Committed: https://code.google.com/p/gyp/source/detail?r=1522

[Evan Martin, 2012-10-22 20:31:04]

this is probably less readable but somehow i feel compelled to comment

https://codereview.chromium.org/11236042/diff/2001/test/lib/TestGyp.py
File test/lib/TestGyp.py (right):

https://codereview.chromium.org/11236042/diff/2001/test/lib/TestGyp.py#newcod...
test/lib/TestGyp.py:625: 'E:\\Program Files (x86)', 'E:\\Program Files']
['%s:\\Program Files%s' % (drive, suffix)
 for drive in ['C', 'D', 'E']
 for suffix in ['', ' (x86)']]

[scottmg, 2012-10-22 20:38:42]

On 2012/10/22 20:31:04, Evan Martin wrote:
> this is probably less readable but somehow i feel compelled to comment

Ya, that code is pretty yuck, how do you feel about C..Z?

> 
> https://codereview.chromium.org/11236042/diff/2001/test/lib/TestGyp.py
> File test/lib/TestGyp.py (right):
> 
>
https://codereview.chromium.org/11236042/diff/2001/test/lib/TestGyp.py#newcod...
> test/lib/TestGyp.py:625: 'E:\\Program Files (x86)', 'E:\\Program Files']
> ['%s:\\Program Files%s' % (drive, suffix)
>  for drive in ['C', 'D', 'E']
>  for suffix in ['', ' (x86)']]

[Nico, 2012-10-23 21:06:28]

lgtm

Do any of the gyp bots have 2012 at all?

[scottmg, 2012-10-23 21:09:04]

On 2012/10/23 21:06:28, Nico wrote:
> lgtm
> 
> Do any of the gyp bots have 2012 at all?

No, not yet. We just got the first one on chromium.fyi, so this is probably bit
premature, but I just wanted to see what sorts of errors might be interesting.