Clean up gn VC++ linker settings

Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.

Committed: https://crrev.com/304172167717e13b6579ecfb7e3fc63338de66ac
Cr-Commit-Position: refs/heads/master@{#383399}

[brucedawson, 2016-03-25 18:48:16]

Description was changed from

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place and
need to be moved/merged. Also, the mini_installer link command removes
executable_config and then triggers LNK4199 warnings so that needs to
be added back in.
==========

to

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.
==========

[brucedawson, 2016-03-25 18:48:16]

brucedawson@chromium.org changed reviewers:
+ brettw@chromium.org

[brettw, 2016-03-25 22:11:54]

lgtm

[brucedawson, 2016-03-25 22:24:32]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2016-03-25 22:24:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1835603002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1835603002/1

[commit-bot: I haz the power, 2016-03-25 23:11:32]

Description was changed from

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.
==========

to

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.
==========

[commit-bot: I haz the power, 2016-03-25 23:11:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-25 23:13:14]

Description was changed from

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.
==========

to

==========
Clean up gn VC++ linker settings

/fastfail and /maxilksize:0x7ff00000 were added to the wrong place in
crrev.com/1816333002 and need to be moved/merged. Also, the
mini_installer link command removes executable_config (which suppresses
LNK4199 warnings) and then triggers those warnings so that suppression
needs to be added back in.

The /maxilksize argument is also changed to be consistent and to avoid
duplication.

This doesn't change build behavior at all, it just suppresses some
warnings and tidies up the configuration.

Committed: https://crrev.com/304172167717e13b6579ecfb7e3fc63338de66ac
Cr-Commit-Position: refs/heads/master@{#383399}
==========

[commit-bot: I haz the power, 2016-03-25 23:13:15]

Patchset 1 (id:??) landed as
https://crrev.com/304172167717e13b6579ecfb7e3fc63338de66ac
Cr-Commit-Position: refs/heads/master@{#383399}