GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True

Committed: https://crrev.com/e40a7ee0074e1bbb954e9bfbf8fabb7f73ec5d01
Cr-Commit-Position: refs/heads/master@{#14644}

[kjellander_webrtc, 2016-10-13 16:47:12]

Description was changed from

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets.

BUG=webrtc:4256
==========

to

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets.

BUG=webrtc:4256
NOTRY=True
==========

[kjellander_webrtc, 2016-10-13 16:47:12]

kjellander@webrtc.org changed reviewers:
+ ehmaldonado@webrtc.org

[kjellander_webrtc, 2016-10-13 16:48:06]

Description was changed from

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets.

BUG=webrtc:4256
NOTRY=True
==========

to

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for
https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True
==========

[ehmaldonado_webrtc, 2016-10-14 00:55:34]

On 2016/10/13 16:49:23, kjellander_webrtc wrote:

LGTM.
I wonder if it will compile in Chromium.
What are the gn errors it generates?

[kjellander_webrtc, 2016-10-17 06:39:37]

On 2016/10/14 00:55:34, ehmaldonado_webrtc wrote:
> On 2016/10/13 16:49:23, kjellander_webrtc wrote:
> 
> LGTM.
> I wonder if it will compile in Chromium.
> What are the gn errors it generates?

I ran Chromium trybots and they passed with this change. I believe it's because
the //build/config/clang:find_bad_constructs config is not enabled for
third_party code, from Chromium's perspective.

[kjellander_webrtc, 2016-10-17 06:39:43]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-10-17 06:39:47]

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

[commit-bot: I haz the power, 2016-10-17 06:56:14]

Description was changed from

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for
https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True
==========

to

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for
https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True
==========

[commit-bot: I haz the power, 2016-10-17 06:56:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-17 06:56:27]

Description was changed from

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for
https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True
==========

to

==========
GN: Exclude suppressions of Chromium Clang warnings for Chromium builds.

These suppressions are causing GN errors when Chromium targets are depending
directly on WebRTC targets (needed for
https://codereview.chromium.org/2413103004)

BUG=webrtc:4256
NOTRY=True

Committed: https://crrev.com/e40a7ee0074e1bbb954e9bfbf8fabb7f73ec5d01
Cr-Commit-Position: refs/heads/master@{#14644}
==========

[commit-bot: I haz the power, 2016-10-17 06:56:28]

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