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

Unified Diff: media/cast/video_sender/video_encoder_unittest.cc

Issue 74133002: Cast: Removed unnecessary ref counters. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed to lock form atomic int Created 7 years, 1 month 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: media/cast/video_sender/video_encoder_unittest.cc
diff --git a/media/cast/video_sender/video_encoder_unittest.cc b/media/cast/video_sender/video_encoder_unittest.cc
index a68db2271cbf7bd55da136ffec2d3b438eec9479..98ea8a9f6e488f0a0f0aa9eccd8c0d5796c5e767 100644
--- a/media/cast/video_sender/video_encoder_unittest.cc
+++ b/media/cast/video_sender/video_encoder_unittest.cc
@@ -101,8 +101,8 @@ class VideoEncoderTest : public ::testing::Test {
}
void Configure(uint8 max_unacked_frames) {
- video_encoder_= new VideoEncoder(cast_environment_, video_config_,
- max_unacked_frames);
+ video_encoder_.reset(new VideoEncoder(cast_environment_, video_config_,
+ max_unacked_frames));
video_encoder_controller_ = video_encoder_.get();
}
@@ -111,7 +111,7 @@ class VideoEncoderTest : public ::testing::Test {
scoped_refptr<TestVideoEncoderCallback> test_video_encoder_callback_;
VideoSenderConfig video_config_;
scoped_refptr<test::FakeTaskRunner> task_runner_;
- scoped_refptr<VideoEncoder> video_encoder_;
+ scoped_ptr<VideoEncoder> video_encoder_;
VideoEncoderController* video_encoder_controller_;
I420VideoFrame video_frame_;

Powered by Google App Engine
This is Rietveld 408576698