media: Introduce Renderer interface and RendererImpl.

media/base/filter_collection.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_FILTER_COLLECTION_H_
 #define MEDIA_BASE_FILTER_COLLECTION_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "media/base/media_export.h"
 
 namespace media {
 
-class AudioRenderer;
 class Demuxer;
+class Renderer;
 class TextRenderer;
-class VideoRenderer;
 
 // Represents a set of uninitialized demuxer and audio/video decoders and

[scherkus (not reviewing), 2014/08/22 20:50:01]

I guess there aren't even decoders ... this whole comment is kinda crummy anyway
haha

[xhwang, 2014/08/22 22:48:22]

Done.

-// renderers. Used to start a Pipeline object for media playback.
+// renderer. Used to start a Pipeline object for media playback.
 //
 // TODO(scherkus): Replace FilterCollection with something sensible, see
 // http://crbug.com/110800
 class MEDIA_EXPORT FilterCollection {
  public:
   FilterCollection();
   ~FilterCollection();
 
   void SetDemuxer(Demuxer* demuxer);
   Demuxer* GetDemuxer();
 
-  void SetAudioRenderer(scoped_ptr<AudioRenderer> audio_renderer);
-  scoped_ptr<AudioRenderer> GetAudioRenderer();
-
-  void SetVideoRenderer(scoped_ptr<VideoRenderer> video_renderer);
-  scoped_ptr<VideoRenderer> GetVideoRenderer();
+  void SetRenderer(scoped_ptr<Renderer> renderer);
+  scoped_ptr<Renderer> GetRenderer();
 
   void SetTextRenderer(scoped_ptr<TextRenderer> text_renderer);
   scoped_ptr<TextRenderer> GetTextRenderer();

[scherkus (not reviewing), 2014/08/22 20:50:01]

considering TextRenderer is concrete, we should consider:
  - Moving switches::kEnableInbandTextTracks to media/
  - Have Pipeline create a TextRenderer if switch is present
  - Removing this from FilterCollection
  - Changing Pipeline ctor to accept a Demuxer and Renderer
  - Deleting FilterCollection and fix crbug.com/110800 once and for all :)

(not in this CL of course ... but it's great post-CL cleanup!)

[xhwang, 2014/08/22 22:48:22]

Modified TODO to reflect this. Will do after this lands.

 
  private:
   Demuxer* demuxer_;
-  scoped_ptr<AudioRenderer> audio_renderer_;
-  scoped_ptr<VideoRenderer> video_renderer_;
+  scoped_ptr<Renderer> renderer_;
   scoped_ptr<TextRenderer> text_renderer_;
 
   DISALLOW_COPY_AND_ASSIGN(FilterCollection);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_FILTER_COLLECTION_H_