media: scaffolding and plumbing for MojoRenderer{Impl, Service}

media/mojo/interfaces/media_renderer.mojom

Index: media/mojo/interfaces/media_renderer.mojom
diff --git a/media/mojo/interfaces/media_renderer.mojom b/media/mojo/interfaces/media_renderer.mojom
index 09b14820ef2f3b8121c6bc5b3db4bdbeb904c52b..555472ef2da71d6ddf7333118ae19b822fd99cda 100644
--- a/media/mojo/interfaces/media_renderer.mojom
+++ b/media/mojo/interfaces/media_renderer.mojom
@@ -2,24 +2,17 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+import "media/mojo/interfaces/demuxer_stream.mojom"
 import "media/mojo/interfaces/media_types.mojom"
 
 module mojo {
 
 [Client=MediaRendererClient]
 interface MediaRenderer {
-  // Initializes the Renderer, calling back upon completion.
+  // Initializes the Renderer with |stream|, calling back upon completion.
   // NOTE: If an error occurs, MediaRendererClient::OnError() will be called
   // before the callback is executed.
-  Initialize() => ();
-
-  // Decodes and renders |buffer|, calling back when more data is required.
-  // NOTE: If an error occurs, MediaRendererClient::OnError() will be called
-  // before the callback is executed.
-  // TODO(tim): Switch decoding model to use framed data pipe when available.
-  // In that world, the signalling for more would be implicit by a writable
-  // pipe handle on the client side so this entire method + callback goes away.
-  DecodeAndRender(MediaDecoderBuffer buffer) => ();
+  Initialize(DemuxerStream stream) => ();

[scherkus (not reviewing), 2014/09/09 20:35:32]

FYI this will likely change in response to
https://codereview.chromium.org/523283002/

[tim (not reviewing), 2014/09/10 23:08:28]

OK... I'll loop back to this once I address the bulk of the comments.

 
   // Discards any buffered data, executing callback when completed.
   // NOTE: If an error occurs, MediaRendererClient::OnError() can be called