Add SetReportBlocks to rtcp Sender/Receive Report classes.

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

Index: webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc b/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc
index 59c08c3c66ef093761f7f32fab79a3efaf2720f4..a43ee18b672a8bdf3147727d1589460c4f963d53 100644
--- a/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc
@@ -10,6 +10,8 @@
 
 #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"
@@ -18,6 +20,7 @@
 namespace webrtc {
 namespace rtcp {
 constexpr uint8_t SenderReport::kPacketType;
+constexpr size_t SenderReport::kMaxNumberOfReportBlocks;

[eladalon, 2017/07/25 13:26:09]

I don't think these two are necessary. Maybe only for certain compilers?
Locally, at least, I don't need this.

[danilchap, 2017/07/25 13:39:00]

it might not be used right now, but when it is needed and absent, error message
might be surprising.

https://github.com/google/googletest/blob/master/googletest/docs/FAQ.md#the-c...

 //    Sender report (SR) (RFC 3550).
 //     0                   1                   2                   3
 //     0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
@@ -122,5 +125,15 @@ bool SenderReport::AddReportBlock(const ReportBlock& block) {
   return true;
 }
 
+bool SenderReport::SetReportBlocks(std::vector<ReportBlock> blocks) {
+  if (blocks.size() > kMaxNumberOfReportBlocks) {
+    LOG(LS_WARNING) << "Too many report blocks (" << blocks.size()
+                    << ") for sender report.";
+    return false;
+  }
+  report_blocks_ = std::move(blocks);
+  return true;
+}
+
 }  // namespace rtcp
 }  // namespace webrtc