Refactor composing report blocks for rtcp Sender/Receiver reports

webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc

 /*
  *  Copyright (c) 2016 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.
  */
 
 #include "webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.h"
 
+#include <utility>
+
 #include "webrtc/modules/rtp_rtcp/source/byte_io.h"
 #include "webrtc/modules/rtp_rtcp/source/rtcp_packet/common_header.h"
 #include "webrtc/rtc_base/checks.h"
 #include "webrtc/rtc_base/logging.h"
 
 namespace webrtc {
 namespace rtcp {
 constexpr uint8_t SenderReport::kPacketType;
 //    Sender report (SR) (RFC 3550).
 //     0                   1                   2                   3
@@ skipping 92 matching lines @@
 
 bool SenderReport::AddReportBlock(const ReportBlock& block) {
   if (report_blocks_.size() >= kMaxNumberOfReportBlocks) {
     LOG(LS_WARNING) << "Max report blocks reached.";
     return false;
   }
   report_blocks_.push_back(block);
   return true;
 }
 
+bool SenderReport::SetReportBlocks(std::vector<ReportBlock> blocks) {

[eladalon, 2017/07/21 12:53:19]

Similar comments for this file and the next one as for the Receiver case.

[danilchap, 2017/07/21 17:11:14]

Done.

+  if (blocks.size() > kMaxNumberOfReportBlocks) {
+    LOG(LS_WARNING) << "Max report blocks reached.";
+    return false;
+  }
+  report_blocks_ = std::move(blocks);
+  return true;
+}
+
 }  // namespace rtcp
 }  // namespace webrtc