Add support for ProgramDatabaseFile to Ninja

Add support for ProgramDatabaseFile to Ninja

Currently we ignore the ProgramDatabaseFile option in the linker
portion of the GYP file.  This change will cause the /PDB:<name>
switch to be emitted during the link step as part of ldflags.
Since ldflags are appended to the defaults, they override any
default /PDB switches that are current emitted which is consistent
with our current uses.

BUG=126499
TEST=gyptest-link-pdb
Committed: https://code.google.com/p/gyp/source/detail?r=1530

[scottmg, 2012-11-02 19:27:01]

lgtm

[iannucci, 2012-11-02 21:09:34]

lgtm, just a typo

https://chromiumcodereview.appspot.com/11359044/diff/1/test/win/linker-flags/...
File test/win/linker-flags/program-database.gyp (right):

https://chromiumcodereview.appspot.com/11359044/diff/1/test/win/linker-flags/...
test/win/linker-flags/program-database.gyp:7: # Verify that 'ProgramDataBase'
option correctly makes it to LINK steup in Ninja
steup -> typo?

[iannucci, 2012-11-02 21:10:05]

On 2012/11/02 21:09:34, iannucci wrote:
> lgtm, just a typo
> 
>
https://chromiumcodereview.appspot.com/11359044/diff/1/test/win/linker-flags/...
> File test/win/linker-flags/program-database.gyp (right):
> 
>
https://chromiumcodereview.appspot.com/11359044/diff/1/test/win/linker-flags/...
> test/win/linker-flags/program-database.gyp:7: # Verify that 'ProgramDataBase'
> option correctly makes it to LINK steup in Ninja
> steup -> typo?

Oops, already committed, I see :)

[noelallen1, 2012-11-03 00:49:41]

http://codereview.chromium.org/11359044/diff/1/test/win/linker-flags/program-...
File test/win/linker-flags/program-database.gyp (right):

http://codereview.chromium.org/11359044/diff/1/test/win/linker-flags/program-...
test/win/linker-flags/program-database.gyp:7: # Verify that 'ProgramDataBase'
option correctly makes it to LINK steup in Ninja
Fixing in new CL:
https://codereview.chromium.org/11368059

On 2012/11/02 21:09:34, iannucci wrote:
> steup -> typo?