Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(595)

Unified Diff: webrtc/call/rtc_event_log_unittest.cc

Issue 2370313002: Reland of Unify rtcp packet setters (Closed)
Patch Set: Fix breaking mistype Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webrtc/modules/pacing/packet_router.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/call/rtc_event_log_unittest.cc
diff --git a/webrtc/call/rtc_event_log_unittest.cc b/webrtc/call/rtc_event_log_unittest.cc
index 34e993cf5cd73290db8e6b2bd2b5e16db1ad39bc..8e92c7e37def5403125737489702105656373fab 100644
--- a/webrtc/call/rtc_event_log_unittest.cc
+++ b/webrtc/call/rtc_event_log_unittest.cc
@@ -139,15 +139,14 @@ RtpPacketToSend GenerateRtpPacket(const RtpHeaderExtensionMap* extensions,
rtc::Buffer GenerateRtcpPacket(Random* prng) {
rtcp::ReportBlock report_block;
- report_block.To(prng->Rand<uint32_t>()); // Remote SSRC.
- report_block.WithFractionLost(prng->Rand(50));
+ report_block.SetMediaSsrc(prng->Rand<uint32_t>()); // Remote SSRC.
+ report_block.SetFractionLost(prng->Rand(50));
rtcp::SenderReport sender_report;
- sender_report.From(prng->Rand<uint32_t>()); // Sender SSRC.
- sender_report.WithNtp(
- NtpTime(prng->Rand<uint32_t>(), prng->Rand<uint32_t>()));
- sender_report.WithPacketCount(prng->Rand<uint32_t>());
- sender_report.WithReportBlock(report_block);
+ sender_report.SetSenderSsrc(prng->Rand<uint32_t>());
+ sender_report.SetNtp(NtpTime(prng->Rand<uint32_t>(), prng->Rand<uint32_t>()));
+ sender_report.SetPacketCount(prng->Rand<uint32_t>());
+ sender_report.AddReportBlock(report_block);
return sender_report.Build();
}
« no previous file with comments | « no previous file | webrtc/modules/pacing/packet_router.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698