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

Unified Diff: webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc

Issue 2110763002: Style updates to ProducerFec/FecReceiver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rename kNumPacketsPerFrameThresholdForMinNumMediaPacketsAdditiveAdaptation Created 4 years, 5 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
Index: webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
diff --git a/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc b/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
index 0c66dc1f528ea68cd787616ddf279948a8cd3dcd..e8ec1f4622478530b45fcda60953b596e8c52be3 100644
--- a/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
+++ b/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
@@ -71,19 +71,19 @@ class ReceiverFecTest : public ::testing::Test {
}
void BuildAndAddRedMediaPacket(test::RawRtpPacket* packet) {
- test::RawRtpPacket* red_packet = generator_->BuildMediaRedPacket(packet);
+ std::unique_ptr<test::RawRtpPacket> red_packet(
+ generator_->BuildMediaRedPacket(packet));
EXPECT_EQ(0, receiver_fec_->AddReceivedRedPacket(
red_packet->header.header, red_packet->data,
red_packet->length, kFecPayloadType));
- delete red_packet;
}
void BuildAndAddRedFecPacket(Packet* packet) {
- test::RawRtpPacket* red_packet = generator_->BuildFecRedPacket(packet);
+ std::unique_ptr<test::RawRtpPacket> red_packet(
+ generator_->BuildFecRedPacket(packet));
EXPECT_EQ(0, receiver_fec_->AddReceivedRedPacket(
red_packet->header.header, red_packet->data,
red_packet->length, kFecPayloadType));
- delete red_packet;
}
void InjectGarbagePacketLength(size_t fec_garbage_offset);

Powered by Google App Engine
This is Rietveld 408576698