Finish plumbing MojoVideoDecoder.

media/mojo/interfaces/video_decoder.mojom

+// 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);
+};