Fix chromium-style warnings.

webrtc/modules/rtp_rtcp/BUILD.gn

Index: webrtc/modules/rtp_rtcp/BUILD.gn
diff --git a/webrtc/modules/rtp_rtcp/BUILD.gn b/webrtc/modules/rtp_rtcp/BUILD.gn
index 16a3717092d64c3eea34c7058c5fc525dd16cb7b..790f99a5bad63244e34c3ecbe14aaa11ce55d1a0 100644
--- a/webrtc/modules/rtp_rtcp/BUILD.gn
+++ b/webrtc/modules/rtp_rtcp/BUILD.gn
@@ -133,6 +133,8 @@ rtc_static_library("rtp_rtcp") {
     "source/rtp_receiver_video.cc",
     "source/rtp_receiver_video.h",
     "source/rtp_rtcp_config.h",
+    "source/rtp_rtcp_defines_nullimpl.cc",

[danilchap, 2016/10/07 12:42:49]

may be call this files
null_rtp_callbacks

[terelius, 2016/10/07 15:03:37]

My reasoning for choosing this name is that the file essentially contains the
null implementations of the interfaces declared in rtcp_defines.h. Since there
are other null objects/callbacks in the RTP module, I prefer a name that
clarifies which objects live in the file. Or do you think we will want to move
in other objects into this file?

[danilchap, 2016/10/07 17:22:00]

might make sense to move the remaining object into that file too eventually.
(found just one another: NullReceiveStatistics).

for now might better to leave rtp_rtcp_defines_nullimpl to simplify lookup

+    "source/rtp_rtcp_defines_nullimpl.h",
     "source/rtp_rtcp_impl.cc",
     "source/rtp_rtcp_impl.h",
     "source/rtp_sender.cc",