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

Unified Diff: media/base/demuxer_perftest.cc

Issue 2491043003: MediaResource refactoring to support multiple streams (Closed)
Patch Set: rebase Created 3 years, 11 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/base/demuxer_perftest.cc
diff --git a/media/base/demuxer_perftest.cc b/media/base/demuxer_perftest.cc
index 6fccde25e1a2a05a77d9a23db922312aa0cbc37f..ccb8d93789fe6b899297bcdeeeb23ce1a52bffc1 100644
--- a/media/base/demuxer_perftest.cc
+++ b/media/base/demuxer_perftest.cc
@@ -102,22 +102,12 @@ class StreamReader {
StreamReader::StreamReader(media::Demuxer* demuxer,
bool enable_bitstream_converter) {
- media::DemuxerStream* stream =
- demuxer->GetStream(media::DemuxerStream::AUDIO);
- if (stream) {
+ std::vector<media::DemuxerStream*> streams = demuxer->GetStreams();
+ for (const auto& stream : streams) {
streams_.push_back(stream);
end_of_stream_.push_back(false);
last_read_timestamp_.push_back(media::kNoTimestamp);
counts_.push_back(0);
- }
-
- stream = demuxer->GetStream(media::DemuxerStream::VIDEO);
- if (stream) {
- streams_.push_back(stream);
- end_of_stream_.push_back(false);
- last_read_timestamp_.push_back(media::kNoTimestamp);
- counts_.push_back(0);
-
if (enable_bitstream_converter)
stream->EnableBitstreamConverter();
tguilbert 2017/01/24 23:24:40 Could there be any problems with enabling this on
servolk 2017/01/25 00:57:32 I believe there shouldn't be any problems enabling
}

Powered by Google App Engine
This is Rietveld 408576698