media: Support RendererFactory in MojoMediaClient.

media/BUILD.gn

Index: media/BUILD.gn
diff --git a/media/BUILD.gn b/media/BUILD.gn
index 3cdcd5c2a3d463ce2febe29552c1171464b805c8..1964ae3392b89890521dd2d4248d9225c9f12246 100644
--- a/media/BUILD.gn
+++ b/media/BUILD.gn
@@ -196,6 +196,8 @@ component("media") {
     "formats/webm/webm_webvtt_parser.h",
     "renderers/audio_renderer_impl.cc",
     "renderers/audio_renderer_impl.h",
+    "renderers/default_renderer_factory.cc",
+    "renderers/default_renderer_factory.h",
     "renderers/gpu_video_accelerator_factories.h",
     "renderers/renderer_impl.cc",
     "renderers/renderer_impl.h",
@@ -348,8 +350,6 @@ component("media") {
     sources += [
       "filters/opus_audio_decoder.cc",
       "filters/opus_audio_decoder.h",

[xhwang, 2015/07/08 16:58:16]

What's the reason we disable opus audio decoder on Android?

[DaleCurtis, 2015/07/08 18:22:48]

I don't think any decoders should be enabled today on android?

[xhwang, 2015/07/08 18:39:09]

There are a lot of classes (e.g. RendererImpl, VideoRendererImpl, etc) that we
don't use on Android but are enabled on all platforms. If they end up not being
used on a platform, the code will be stripped out. So it seems to me we should
only disable code that will NOT work on a platform, not code that is not enabled
or needed on/by a platform?

[DaleCurtis, 2015/07/08 18:44:16]

If by not used you mean they will not be compiled via -= [sources] blocks then
that's fine with me. I don't think we should rely on the linker to strip dead
unused code post-compilation though.

-      "renderers/default_renderer_factory.cc",
-      "renderers/default_renderer_factory.h",
     ]
   }