Address some violations of chromium-style.

webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 366 matching lines @@
   virtual void OnRttUpdate(int64_t rtt) = 0;
 
   virtual int64_t LastProcessedRtt() const = 0;
 
   virtual ~RtcpRttStats() {}
 };
 
 // Null object version of RtpFeedback.
 class NullRtpFeedback : public RtpFeedback {
  public:
-  virtual ~NullRtpFeedback() {}
+  ~NullRtpFeedback() override {}
 
   int32_t OnInitializeDecoder(int8_t payload_type,
                               const char payloadName[RTP_PAYLOAD_NAME_SIZE],
                               int frequency,
                               size_t channels,
-                              uint32_t rate) override {
-    return 0;
-  }
+                              uint32_t rate) override;
 
   void OnIncomingSSRCChanged(uint32_t ssrc) override {}
   void OnIncomingCSRCChanged(uint32_t csrc, bool added) override {}
 };
 
+inline int32_t NullRtpFeedback::OnInitializeDecoder(

[kwiberg-webrtc, 2017/05/31 08:22:28]

I'm not sure I like the use of "inline" here---you're basically just tricking
the style checker by using a construct that it should warn about, but it doesn't
because it's obscure enough that that warning hasn't been implemented.

I'd prefer if you simply made a new .cc file to go with this .h file, but I know
that can be a bunch of icky work, so I won't insist.

[nisse-webrtc, 2017/05/31 08:44:54]

I'm not sure what's the background for the warning about inline, is it about
readability or about binary size?
  
I'd prefer not to create a new .cc file in this particular case, because I'd
prefer if these NullFoo classes were just deleted.

[kwiberg-webrtc, 2017/05/31 08:49:48]

Binary size. It warns about inlined constructors, destructors, and virtual
functions, presumably because they were common causes of unnecessary binary
bloat.
OK, fair enough.

+    int8_t payload_type,
+    const char payloadName[RTP_PAYLOAD_NAME_SIZE],
+    int frequency,
+    size_t channels,
+    uint32_t rate) {
+  return 0;
+}
+
 // Null object version of RtpData.
 class NullRtpData : public RtpData {
  public:
-  virtual ~NullRtpData() {}
+  ~NullRtpData() override {}
 
   int32_t OnReceivedPayloadData(const uint8_t* payload_data,
                                 size_t payload_size,
-                                const WebRtcRTPHeader* rtp_header) override {
-    return 0;
-  }
+                                const WebRtcRTPHeader* rtp_header) override;
 };
 
+inline int32_t NullRtpData::OnReceivedPayloadData(
+    const uint8_t* payload_data,
+    size_t payload_size,
+    const WebRtcRTPHeader* rtp_header) {
+  return 0;
+}
+
 // Statistics about packet loss for a single directional connection. All values
 // are totals since the connection initiated.
 struct RtpPacketLossStats {
   // The number of packets lost in events where no adjacent packets were also
   // lost.
   uint64_t single_packet_loss_count;
   // The number of events in which more than one adjacent packet was lost.
   uint64_t multiple_packet_loss_event_count;
   // The number of packets lost in events where more than one adjacent packet
   // was lost.
@@ skipping 26 matching lines @@
 class TransportSequenceNumberAllocator {
  public:
   TransportSequenceNumberAllocator() {}
   virtual ~TransportSequenceNumberAllocator() {}
 
   virtual uint16_t AllocateSequenceNumber() = 0;
 };
 
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_DEFINES_H_