Add gyp for bench and make run on Windows.

[epoger, 2011-05-31 20:18:02]

http://codereview.appspot.com/4528112/diff/1/gyp/bench.gyp
File gyp/bench.gyp (right):

http://codereview.appspot.com/4528112/diff/1/gyp/bench.gyp#newcode9
gyp/bench.gyp:9: 'mac_bundle' : 1,
I would take out the mac_bundle line for now.  I don't know if it's really
needed for Mac or not, and if it is, I'd like to see if we can put it somewhere
common instead of in each target separately.

http://codereview.appspot.com/4528112/diff/1/gyp/bench.gyp#newcode10
gyp/bench.gyp:10: 'include_dirs': [
I think you can safely remove the entire include_dirs stanza (the dependencies
section provides these includes already)

http://codereview.appspot.com/4528112/diff/1/gyp/bench.gyp#newcode31
gyp/bench.gyp:31: 'core.gyp:core',
please put these in alpha order

http://codereview.appspot.com/4528112/diff/1/gyp/bench.gyp#newcode38
gyp/bench.gyp:38: 'msvs_settings': {
Please tell me (in person) what you've figured out about msvs_settings.  How
sensitive are we to the individual settings?  Can we put this in some common
place so all Windows targets inherit it?

http://codereview.appspot.com/4528112/diff/1/gyp/core.gyp
File gyp/core.gyp (right):

http://codereview.appspot.com/4528112/diff/1/gyp/core.gyp#newcode308
gyp/core.gyp:308: '../src/ports/SkTime_win.cpp',
Please tell me again why it's better to add this and then conditionally remove
it, rather than just conditionally adding it?

[epoger, 2011-05-31 20:50:13]

LGTM

[bungeman, 2011-05-31 21:22:46]

Committed revision 1458.