Revert of Unify rtcp packet setters

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

Index: webrtc/modules/rtp_rtcp/source/rtcp_packet/bye_unittest.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_packet/bye_unittest.cc b/webrtc/modules/rtp_rtcp/source/rtcp_packet/bye_unittest.cc
index c90566b98de5537650a9b627f11589ccf540c5e5..fbea60a737775b51cab0b46dce525d70b828e683 100644
--- a/webrtc/modules/rtp_rtcp/source/rtcp_packet/bye_unittest.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtcp_packet/bye_unittest.cc
@@ -26,7 +26,7 @@
 
 TEST(RtcpPacketByeTest, CreateAndParseWithoutReason) {
   Bye bye;
-  bye.SetSenderSsrc(kSenderSsrc);
+  bye.From(kSenderSsrc);
 
   rtc::Buffer raw = bye.Build();
   Bye parsed_bye;
@@ -39,8 +39,9 @@
 
 TEST(RtcpPacketByeTest, CreateAndParseWithCsrcs) {
   Bye bye;
-  bye.SetSenderSsrc(kSenderSsrc);
-  EXPECT_TRUE(bye.SetCsrcs({kCsrc1, kCsrc2}));
+  bye.From(kSenderSsrc);
+  EXPECT_TRUE(bye.WithCsrc(kCsrc1));
+  EXPECT_TRUE(bye.WithCsrc(kCsrc2));
   EXPECT_TRUE(bye.reason().empty());
 
   rtc::Buffer raw = bye.Build();
@@ -56,9 +57,10 @@
   Bye bye;
   const std::string kReason = "Some Reason";
 
-  bye.SetSenderSsrc(kSenderSsrc);
-  EXPECT_TRUE(bye.SetCsrcs({kCsrc1, kCsrc2}));
-  bye.SetReason(kReason);
+  bye.From(kSenderSsrc);
+  EXPECT_TRUE(bye.WithCsrc(kCsrc1));
+  EXPECT_TRUE(bye.WithCsrc(kCsrc2));
+  bye.WithReason(kReason);
 
   rtc::Buffer raw = bye.Build();
   Bye parsed_bye;
@@ -71,18 +73,20 @@
 
 TEST(RtcpPacketByeTest, CreateWithTooManyCsrcs) {
   Bye bye;
-  bye.SetSenderSsrc(kSenderSsrc);
+  bye.From(kSenderSsrc);
   const int kMaxCsrcs = (1 << 5) - 2;  // 5 bit len, first item is sender SSRC.
-  EXPECT_TRUE(bye.SetCsrcs(std::vector<uint32_t>(kMaxCsrcs, kCsrc1)));
-  EXPECT_FALSE(bye.SetCsrcs(std::vector<uint32_t>(kMaxCsrcs + 1, kCsrc1)));
+  for (int i = 0; i < kMaxCsrcs; ++i) {
+    EXPECT_TRUE(bye.WithCsrc(i));
+  }
+  EXPECT_FALSE(bye.WithCsrc(kMaxCsrcs));
 }
 
 TEST(RtcpPacketByeTest, CreateAndParseWithAReason) {
   Bye bye;
   const std::string kReason = "Some Random Reason";
 
-  bye.SetSenderSsrc(kSenderSsrc);
-  bye.SetReason(kReason);
+  bye.From(kSenderSsrc);
+  bye.WithReason(kReason);
 
   rtc::Buffer raw = bye.Build();
   Bye parsed_bye;
@@ -99,8 +103,8 @@
   for (size_t reminder = 0; reminder < 4; ++reminder) {
     const std::string kReason(4 + reminder, 'a' + reminder);
     Bye bye;
-    bye.SetSenderSsrc(kSenderSsrc);
-    bye.SetReason(kReason);
+    bye.From(kSenderSsrc);
+    bye.WithReason(kReason);
 
     rtc::Buffer raw = bye.Build();
     Bye parsed_bye;
@@ -121,7 +125,7 @@
 
 TEST(RtcpPacketByeTest, ParseFailOnInvalidSrcCount) {
   Bye bye;
-  bye.SetSenderSsrc(kSenderSsrc);
+  bye.From(kSenderSsrc);
 
   rtc::Buffer raw = bye.Build();
   raw[0]++;  // Damage the packet: increase ssrc count by one.
@@ -132,8 +136,8 @@
 
 TEST(RtcpPacketByeTest, ParseFailOnInvalidReasonLength) {
   Bye bye;
-  bye.SetSenderSsrc(kSenderSsrc);
-  bye.SetReason("18 characters long");
+  bye.From(kSenderSsrc);
+  bye.WithReason("18 characters long");
 
   rtc::Buffer raw = bye.Build();
   // Damage the packet: decrease payload size by 4 bytes