Enable //build/config/clang:extra_warnings for rtc_media

Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True

Committed: https://crrev.com/660312b0c6add8e14b2a1a908f24bf260ae6453a
Cr-Commit-Position: refs/heads/master@{#14381}

[kjellander_webrtc, 2016-09-26 10:00:32]

Description was changed from

==========
Enable //build/config/clang:extra_warnings for rtc_media

It seems this actually works to have enabled.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

to

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles with without removing this config.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

[kjellander_webrtc, 2016-09-26 10:00:33]

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

[ehmaldonado_webrtc, 2016-09-26 11:35:24]

Description was changed from

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles with without removing this config.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

to

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles without removing this config.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

[ehmaldonado_webrtc, 2016-09-26 11:35:38]

On 2016/09/26 11:33:50, kjellander_webrtc wrote:

LGTM

[kjellander_webrtc, 2016-09-26 13:04:53]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-09-26 13:04:57]

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

[kjellander_webrtc, 2016-09-26 13:06:46]

Description was changed from

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles without removing this config.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

to

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

[commit-bot: I haz the power, 2016-09-26 13:11:56]

Description was changed from

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

to

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

[commit-bot: I haz the power, 2016-09-26 13:11:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-26 13:12:04]

Description was changed from

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True
==========

to

==========
Enable //build/config/clang:extra_warnings for rtc_media

The code compiles fine with this config removed.

BUG=webrtc:6323
NOTRY=True
NOTREECHECKS=True

Committed: https://crrev.com/660312b0c6add8e14b2a1a908f24bf260ae6453a
Cr-Commit-Position: refs/heads/master@{#14381}
==========

[commit-bot: I haz the power, 2016-09-26 13:12:05]

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