Drop return value from RtpRtcp::IncomingRtcpPacket.

webrtc/modules/rtp_rtcp/test/testAPI/test_api.cc

 /*
  *  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 44 matching lines @@
   }
   const uint8_t* payload = data + header.headerLength;
   RTC_CHECK_GE(len, header.headerLength);
   const size_t payload_length = len - header.headerLength;
   receive_statistics_->IncomingPacket(header, len, false);
   return rtp_receiver_->IncomingRtpPacket(header, payload, payload_length,
                                           payload_specific, true);
 }
 
 bool LoopBackTransport::SendRtcp(const uint8_t* data, size_t len) {
-  if (rtp_rtcp_module_->IncomingRtcpPacket((const uint8_t*)data, len) < 0) {
-    return false;
-  }
+  rtp_rtcp_module_->IncomingRtcpPacket((const uint8_t*)data, len);

[danilchap, 2017/09/13 11:19:13]

mind removing unneeded c-cast while touching this line?

   return true;
 }
 
 int32_t TestRtpReceiver::OnReceivedPayloadData(
     const uint8_t* payload_data,
     size_t payload_size,
     const webrtc::WebRtcRTPHeader* rtp_header) {
   EXPECT_LE(payload_size, sizeof(payload_data_));
   memcpy(payload_data_, payload_data, payload_size);
   memcpy(&rtp_header_, rtp_header, sizeof(rtp_header_));
@@ skipping 97 matching lines @@
   rtx_header.payloadType = kRtxPayloadType;
   EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
   rtx_header.ssrc = 0;
   EXPECT_FALSE(rtp_payload_registry_->IsRtx(rtx_header));
   rtx_header.ssrc = kRtxSsrc;
   rtx_header.payloadType = 0;
   EXPECT_TRUE(rtp_payload_registry_->IsRtx(rtx_header));
 }
 
 }  // namespace webrtc