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

Unified Diff: media/filters/video_frame_stream_unittest.cc

Issue 2871503002: Remove ScopedVector from audio/video renderer related code in media/ (Closed)
Patch Set: Remove ScopedVector from audio/video render related code in media/ Created 3 years, 7 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: media/filters/video_frame_stream_unittest.cc
diff --git a/media/filters/video_frame_stream_unittest.cc b/media/filters/video_frame_stream_unittest.cc
index 2baf2347be21a6a1f3fd4e873dc535cfd2c79bab..3e7873e7f4b3e43bcd4fe1ffeba8eaf821fc4104 100644
--- a/media/filters/video_frame_stream_unittest.cc
+++ b/media/filters/video_frame_stream_unittest.cc
@@ -124,7 +124,7 @@ class VideoFrameStreamTest
// decoder_->SimulateFailureToInit(), and
// - on decode error of the first buffer, which can be simulated by calling
// decoder_->SimulateError() before reading the first frame.
- ScopedVector<VideoDecoder> CreateVideoDecodersForTest() {
+ std::vector<std::unique_ptr<VideoDecoder>> CreateVideoDecodersForTest() {
// Previously decoders could have been destroyed on decoder reselection.
decoders_.clear();
@@ -132,22 +132,22 @@ class VideoFrameStreamTest
// TODO(xhwang): We should test the case where only certain decoder
// supports encrypted streams. Currently this is hard to test because we use
// parameterized tests which need to pass in all combinations.
- ScopedVector<VideoDecoder> decoders;
+ std::vector<std::unique_ptr<VideoDecoder>> decoders;
for (int i = 0; i < 3; ++i) {
- FakeVideoDecoder* decoder =
- new FakeVideoDecoder(GetDecoderName(i), GetParam().decoding_delay,
- GetParam().parallel_decoding,
- base::Bind(&VideoFrameStreamTest::OnBytesDecoded,
- base::Unretained(this)));
+ auto decoder = base::MakeUnique<FakeVideoDecoder>(
+ GetDecoderName(i), GetParam().decoding_delay,
+ GetParam().parallel_decoding,
+ base::Bind(&VideoFrameStreamTest::OnBytesDecoded,
+ base::Unretained(this)));
if (GetParam().is_encrypted && !GetParam().has_decryptor)
decoder->EnableEncryptedConfigSupport();
- decoders.push_back(decoder);
-
// Keep a copy of the raw pointers so we can change the behavior of each
// decoder.
- decoders_.push_back(decoder);
+ decoders_.push_back(decoder.get());
+
+ decoders.push_back(std::move(decoder));
}
for (const auto& i : decoder_indices_to_fail_init_)

Powered by Google App Engine
This is Rietveld 408576698