[gn] add missing suppressions for linker warnings on windows

[gn] add missing suppressions for linker warnings on windows

When doing a component build, some test binaries link against the object
files directly, bypassing the components. This results, however, and
rightly so, in linker warnings. In gyp, we just suppressed them. During
the transition to gn, this was dropped for two binaries.

Here I add the suppressions back in.

Long term, we should either change the tests to go through the public
API, or export the required symbols.

BUG=chromium:633688
R=jkummerow@chromium.org

Committed: https://crrev.com/24cb21e327645f0708dadb8abac473b6f09952cf
Cr-Commit-Position: refs/heads/master@{#38793}

[jochen (gone - plz use gerrit), 2016-08-22 15:26:06]

The CQ bit was checked by jochen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-22 15:26:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 15:31:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-22 15:31:08]

Dry run: Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/22276)

[Jakob Kummerow, 2016-08-22 15:38:57]

lgtm

[jochen (gone - plz use gerrit), 2016-08-22 15:45:38]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-08-22 15:45:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2261123003/#ps20001
(title: "updates")

[commit-bot: I haz the power, 2016-08-22 15:45:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 15:48:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-22 15:48:18]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/22282)

[jochen (gone - plz use gerrit), 2016-08-22 16:47:32]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-08-22 16:47:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 16:50:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-22 16:50:46]

Description was changed from

==========
[gn] add missing suppressions for linker warnings on windows

When doing a component build, some test binaries link against the object
files directly, bypassing the components. This results, however, and
rightly so, in linker warnings. In gyp, we just suppressed them. During
the transition to gn, this was dropped for two binaries.

Here I add the suppressions back in.

Long term, we should either change the tests to go through the public
API, or export the required symbols.

BUG=chromium:633688
R=jkummerow@chromium.org
==========

to

==========
[gn] add missing suppressions for linker warnings on windows

When doing a component build, some test binaries link against the object
files directly, bypassing the components. This results, however, and
rightly so, in linker warnings. In gyp, we just suppressed them. During
the transition to gn, this was dropped for two binaries.

Here I add the suppressions back in.

Long term, we should either change the tests to go through the public
API, or export the required symbols.

BUG=chromium:633688
R=jkummerow@chromium.org

Committed: https://crrev.com/24cb21e327645f0708dadb8abac473b6f09952cf
Cr-Commit-Position: refs/heads/master@{#38793}
==========

[commit-bot: I haz the power, 2016-08-22 16:50:47]

Patchset 2 (id:??) landed as
https://crrev.com/24cb21e327645f0708dadb8abac473b6f09952cf
Cr-Commit-Position: refs/heads/master@{#38793}