Finish plumbing MojoVideoDecoder.

media/mojo/interfaces/video_decoder.mojom

Index: media/mojo/interfaces/video_decoder.mojom
diff --git a/media/mojo/interfaces/video_decoder.mojom b/media/mojo/interfaces/video_decoder.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..b0c49f01f649cb123e53566134998cd562dea65f
--- /dev/null
+++ b/media/mojo/interfaces/video_decoder.mojom
@@ -0,0 +1,22 @@
+// Copyright 2016 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.
+
+module media.interfaces;
+
+import "media/mojo/interfaces/media_types.mojom";
+
+interface VideoDecoder {
+  // TODO(sandersd): |cdm_context|.
+  // TODO(sandersd): Return values for NeedsBitstreamConversion(), et. al.
+  Initialize(VideoDecoderClient client, handle<data_pipe_consumer> decoder_buffer_consumer);
+
+  // TODO(sandersd): Do we need to return a failure status?

[xhwang, 2016/04/20 21:09:27]

Yes, we should at least have the equivalent of the init_cb?

[sandersd (OOO until July 31), 2016/04/20 21:16:07]

We do with this setup, there is a callback with no parameters, and on failure we
can close the pipe.

+  Configure(VideoDecoderConfig config, bool low_delay) => ();

[xhwang, 2016/04/20 21:09:27]

I am a bit confused here. The parameters in Initialize() and Configure() both
belongs to C++ VideoDecoder::Initialize() (indirectly).

Where are the gpu specific parameters you plan to pass in?

[sandersd (OOO until July 31), 2016/04/20 21:16:07]

Working on that right now, they will be passed to Initialize().

+  Decode(DecoderBuffer buffer) => (DecodeStatus status);
+  Reset() => ();
+};
+
+interface VideoDecoderClient {
+  OnOutput(VideoFrame frame);
+};