[rtp_rtcp] tmmbr helper simplified by moving set_to_send member out

webrtc/modules/rtp_rtcp/source/rtcp_sender.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 630 matching lines @@
   request.set_ssrc(remote_ssrc_);
   request.set_bitrate_bps(tmmbr_send_ * 1000);
   request.set_packet_overhead(packet_oh_send_);
   tmmbr->WithTmmbr(request);
 
   return rtc::scoped_ptr<rtcp::Tmmbr>(tmmbr);
 }
 
 rtc::scoped_ptr<rtcp::RtcpPacket> RTCPSender::BuildTMMBN(
     const RtcpContext& ctx) {
-  TMMBRSet* boundingSet = tmmbr_help_.BoundingSetToSend();
-  if (boundingSet == nullptr)
-    return nullptr;
-
   rtcp::Tmmbn* tmmbn = new rtcp::Tmmbn();
   tmmbn->From(ssrc_);
-  for (uint32_t i = 0; i < boundingSet->lengthOfSet(); i++) {
-    if (boundingSet->Tmmbr(i) > 0) {
-      tmmbn->WithTmmbr(boundingSet->Ssrc(i), boundingSet->Tmmbr(i),
-                       boundingSet->PacketOH(i));
+  for (const rtcp::TmmbItem& timber : tmmbn_to_send_) {
+    if (timber.bitrate_bps() > 0) {
+      tmmbn->WithTmmbr(timber);
     }
   }
 
   return rtc::scoped_ptr<rtcp::Tmmbn>(tmmbn);
 }
 
 rtc::scoped_ptr<rtcp::RtcpPacket> RTCPSender::BuildAPP(const RtcpContext& ctx) {
   rtcp::App* app = new rtcp::App();
   app->From(ssrc_);
   app->WithSubType(app_sub_type_);
@@ skipping 304 matching lines @@
   CriticalSectionScoped lock(critical_section_rtcp_sender_.get());
   xr_send_receiver_reference_time_enabled_ = enable;
 }
 
 bool RTCPSender::RtcpXrReceiverReferenceTime() const {
   CriticalSectionScoped lock(critical_section_rtcp_sender_.get());
   return xr_send_receiver_reference_time_enabled_;
 }
 
 // no callbacks allowed inside this function
-int32_t RTCPSender::SetTMMBN(const TMMBRSet* boundingSet) {
+void RTCPSender::SetTMMBN(const TMMBRSet* bounding_set) {
   CriticalSectionScoped lock(critical_section_rtcp_sender_.get());
-
-  if (0 == tmmbr_help_.SetTMMBRBoundingSetToSend(boundingSet)) {
-    SetFlag(kRtcpTmmbn, true);
-    return 0;
+  if (!bounding_set) {
+    tmmbn_to_send_.clear();
+  } else {
+    tmmbn_to_send_ = *bounding_set;
   }
-  return -1;
+  SetFlag(kRtcpTmmbn, true);
 }
 
 void RTCPSender::SetFlag(RTCPPacketType type, bool is_volatile) {
   report_flags_.insert(ReportFlag(type, is_volatile));
 }
 
 void RTCPSender::SetFlags(const std::set<RTCPPacketType>& types,
                           bool is_volatile) {
   for (RTCPPacketType type : types)
     SetFlag(type, is_volatile);
@@ skipping 44 matching lines @@
     // RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(Sender);
     // but we can't because of an incorrect warning (C4822) in MVS 2013.
   } sender(transport_, event_log_);
 
   uint8_t buffer[IP_PACKET_SIZE];
   return packet.BuildExternalBuffer(buffer, IP_PACKET_SIZE, &sender) &&
          !sender.send_failure_;
 }
 
 }  // namespace webrtc