Introduce media::RendererFactory interface.

media/filters/default_renderer_factory.h

Index: media/filters/default_renderer_factory.h
diff --git a/media/filters/default_renderer_factory.h b/media/filters/default_renderer_factory.h
new file mode 100644
index 0000000000000000000000000000000000000000..f7806a602063885ae91b56df1d6825b224ff8b61
--- /dev/null
+++ b/media/filters/default_renderer_factory.h
@@ -0,0 +1,45 @@
+// Copyright 2014 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_FILTERS_DEFAULT_RENDERER_FACTORY_H_
+#define MEDIA_FILTERS_DEFAULT_RENDERER_FACTORY_H_
+
+#include "base/callback.h"
+#include "media/base/media_export.h"
+#include "media/base/renderer_factory.h"
+
+namespace media {
+
+class AudioHardwareConfig;
+class AudioRendererSink;
+class GpuVideoAcceleratorFactories;
+class MediaLog;
+
+// The default factory class for creating RendererImpl.
+class MEDIA_EXPORT DefaultRendererFactory : public RendererFactory {

[gunsch, 2014/12/09 04:26:09]

naming nit: aren't specific implementations of interfaces usually named with the
qualifier afterwards? e.g. RendererFactoryDefault rather than
DefaultRendererFactory. (keeps the related files together)

[xhwang, 2014/12/09 18:05:52]

Hmm, I don't think we are following this model in media/ code. For example, we
have FFmpegVideoDecoder and VpxVideoDecoder...

[gunsch, 2014/12/09 18:08:29]

Ah yeah, I guess that's not really consistent codebase-wide. Seems to be more
for platform things (Thing, ThingAndroid, ThingLinux, etc.)

+ public:
+  DefaultRendererFactory(
+      const scoped_refptr<MediaLog>& media_log,
+      const scoped_refptr<GpuVideoAcceleratorFactories>& gpu_factories,
+      const AudioHardwareConfig& audio_hardware_config);
+  ~DefaultRendererFactory() final;
+
+  scoped_ptr<Renderer> CreateRenderer(
+      const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
+      AudioRendererSink* audio_renderer_sink) final;
+
+ private:
+  scoped_refptr<MediaLog> media_log_;
+
+  // Factories for supporting video accelerators. May be null.
+  scoped_refptr<GpuVideoAcceleratorFactories> gpu_factories_;
+
+  const AudioHardwareConfig& audio_hardware_config_;
+
+  DISALLOW_COPY_AND_ASSIGN(DefaultRendererFactory);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_FILTERS_DEFAULT_RENDERER_FACTORY_H_