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

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: Response to feedback. 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 39b5ed86b16daac2d2c27b4bb9ba02bc09e419ce..03f48904aa9c79f15b3fbffe0959a9228f7eb081 100644
--- a/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
+++ b/webrtc/modules/rtp_rtcp/source/fec_receiver_unittest.cc
@@ -73,19 +73,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