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

Unified Diff: media/filters/ffmpeg_demuxer.h

Issue 100085: Adds a second vector of streams to FFmpegDemuxer to handle re-mapped audio/video streams. (Closed)
Patch Set: Created 11 years, 8 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
« no previous file with comments | « no previous file | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_demuxer.h
diff --git a/media/filters/ffmpeg_demuxer.h b/media/filters/ffmpeg_demuxer.h
index 2c113bee1dd5b3c3ac19fbf7a184fa858e34e3d3..640f9cc0997c3ef33e33346697fbe572575b9beb 100644
--- a/media/filters/ffmpeg_demuxer.h
+++ b/media/filters/ffmpeg_demuxer.h
@@ -149,9 +149,17 @@ class FFmpegDemuxer : public Demuxer {
// Latest timestamp read on the demuxer thread.
base::TimeDelta current_timestamp_;
- // Vector of streams.
+ // Two vector of streams:
+ // - |streams_| is indexed for the Demuxer interface GetStream(), which only
+ // contains supported streams and no NULL entries.
+ // - |packet_streams_| is indexed to mirror AVFormatContext when dealing
+ // with AVPackets returned from av_read_frame() and contain NULL entries
+ // representing unsupported streams where we throw away the data.
+ //
+ // Ownership is handled via reference counting.
typedef std::vector< scoped_refptr<FFmpegDemuxerStream> > StreamVector;
StreamVector streams_;
+ StreamVector packet_streams_;
// Thread handle.
base::Thread thread_;
« no previous file with comments | « no previous file | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698