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

Unified Diff: webrtc/modules/video_coding/video_packet_buffer_unittest.cc

Issue 2199133004: PacketBuffer is now ref counted. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: PacketBuffer/RtpFrameReferenceFinder can now be stopped. Created 4 years, 4 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/video_coding/video_packet_buffer_unittest.cc
diff --git a/webrtc/modules/video_coding/video_packet_buffer_unittest.cc b/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
index ee9a853a246271164d8f95a6717446550c826f1e..a7c3b43cba2dde19877ce2d19ddf72460003231a 100644
--- a/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
+++ b/webrtc/modules/video_coding/video_packet_buffer_unittest.cc
@@ -29,7 +29,7 @@ class TestPacketBuffer : public ::testing::Test,
TestPacketBuffer()
: rand_(0x8739211),
clock_(new SimulatedClock(0)),
- packet_buffer_(new PacketBuffer(clock_.get(),
+ packet_buffer_(PacketBuffer::Create(clock_.get(),
kStartSize,
kMaxSize,
this)),
@@ -253,7 +253,7 @@ class TestPacketBuffer : public ::testing::Test,
Random rand_;
std::unique_ptr<Clock> clock_;
- std::unique_ptr<PacketBuffer> packet_buffer_;
+ rtc::scoped_refptr<PacketBuffer> packet_buffer_;
struct FrameComp {
bool operator()(const std::pair<uint16_t, uint8_t> f1,
const std::pair<uint16_t, uint8_t> f2) const {
@@ -934,7 +934,7 @@ TEST_F(TestPacketBuffer, Vp8LayerSync) {
}
TEST_F(TestPacketBuffer, Vp8InsertLargeFrames) {
- packet_buffer_.reset(new PacketBuffer(clock_.get(), 1 << 3, 1 << 12, this));
+ packet_buffer_ = PacketBuffer::Create(clock_.get(), 1 << 3, 1 << 12, this);
uint16_t pid = Rand();
uint16_t seq_num = Rand();

Powered by Google App Engine
This is Rietveld 408576698