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

Unified Diff: media/base/filter_collection.h

Issue 9325044: Remove AudioDecoder from the Filter heirarchy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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/filter_collection.h
diff --git a/media/base/filter_collection.h b/media/base/filter_collection.h
index 35bd1deb86c0871f675e7c69bc73ad3b2eced17c..84a2000a0724ad15badce5e3163d8f28afecb71c 100644
--- a/media/base/filter_collection.h
+++ b/media/base/filter_collection.h
@@ -8,6 +8,7 @@
#include <list>
#include "base/memory/ref_counted.h"
+#include "media/base/audio_decoder.h"
#include "media/base/demuxer_factory.h"
#include "media/base/filters.h"
@@ -26,7 +27,7 @@ class MEDIA_EXPORT FilterCollection {
// Adds a filter to the collection.
void AddVideoDecoder(VideoDecoder* filter);
- void AddAudioDecoder(AudioDecoder* filter);
+ void AddAudioDecoder(AudioDecoder* audio_decoder);
void AddVideoRenderer(VideoRenderer* filter);
void AddAudioRenderer(AudioRenderer* filter);
@@ -41,7 +42,7 @@ class MEDIA_EXPORT FilterCollection {
// If a filter is returned it is removed from the collection.
// Filters are selected in FIFO order.
void SelectVideoDecoder(scoped_refptr<VideoDecoder>* filter_out);
- void SelectAudioDecoder(scoped_refptr<AudioDecoder>* filter_out);
+ void SelectAudioDecoder(scoped_refptr<AudioDecoder>* out);
void SelectVideoRenderer(scoped_refptr<VideoRenderer>* filter_out);
void SelectAudioRenderer(scoped_refptr<AudioRenderer>* filter_out);
@@ -50,7 +51,6 @@ class MEDIA_EXPORT FilterCollection {
// the following types. This is used to mark, identify, and support
// downcasting of different filter types stored in the filters_ list.
enum FilterType {
- AUDIO_DECODER,
VIDEO_DECODER,
AUDIO_RENDERER,
VIDEO_RENDERER,
@@ -61,6 +61,7 @@ class MEDIA_EXPORT FilterCollection {
typedef std::list<FilterListElement> FilterList;
FilterList filters_;
scoped_ptr<DemuxerFactory> demuxer_factory_;
+ std::list<scoped_refptr<AudioDecoder> > audio_decoders_;
// Helper function that adds a filter to the filter list.
void AddFilter(FilterType filter_type, Filter* filter);

Powered by Google App Engine
This is Rietveld 408576698