Add a stub implementation for Mojo AudioDecoder interface

media/mojo/services/mojo_audio_decoder_service.h

+// 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.
+
+#ifndef MEDIA_MOJO_SERVICES_MOJO_AUDIO_DECODER_SERVICE_H_
+#define MEDIA_MOJO_SERVICES_MOJO_AUDIO_DECODER_SERVICE_H_
+
+#include "base/macros.h"
+//#include "base/memory/ref_counted.h"

[xhwang, 2016/03/16 04:12:18]

fix the "//" here.

[Tima Vaisburd, 2016/03/16 19:23:20]

Done.

+#include "base/memory/scoped_ptr.h"
+//#include "base/memory/weak_ptr.h"

[xhwang, 2016/03/16 04:12:18]

fix the "//" here.

[Tima Vaisburd, 2016/03/16 19:23:20]

Done.

+#include "media/mojo/interfaces/audio_decoder.mojom.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
+
+namespace media {
+
+class MojoAudioDecoderService : public interfaces::AudioDecoder {
+ public:
+  MojoAudioDecoderService(
+      scoped_ptr<AudioDecoder> decoder,

[xhwang, 2016/03/16 04:21:58]

BTW, I assume you'll create the AudioDecoder in AndroidMojoMediaClient, and pass
the |decoder| here.

In the next CL, in MojoMediaClient you'll add a call CreateAudioDecoder() and
implement it in AndrdoiMojoMediaClient.

Note that CreateAudioDecoder() returns nullptr by default. So the proxy side
needs to handle this. Usually this results in a connection error. Here's an
example how to handle it:

https://code.google.com/p/chromium/codesearch#chromium/src/media/mojo/service...

[Tima Vaisburd, 2016/03/16 19:23:20]

In the doc you mentioned ServiceFactoryImpl. Should it delegate everything to
MojoMediaClient? I put this note in the doc.

[xhwang, 2016/03/16 19:51:41]

Sorry for the confusion. Yes, platform specific implementations should go to
MojoMediaClient.

+      mojo::InterfaceRequest<interfaces::AudioDecoder> request);
+
+  ~MojoAudioDecoderService() final;
+
+  // interfaces::AudioDecoder implementation
+  void Initialize(interfaces::AudioDecoderClientPtr client,
+                  interfaces::AudioDecoderConfigPtr config,
+                  int32_t cdm_id,
+                  const InitializeCallback& callback) final;
+
+  void Decode(interfaces::DecoderBufferPtr buffer,
+              const DecodeCallback& callback) final;
+
+  void Reset(const ResetCallback& callback) final;
+
+ private:
+  // A binding represents the association between the service and the
+  // communication channel, i.e. the pipe.
+  mojo::StrongBinding<interfaces::AudioDecoder> binding_;
+
+  // The AudioDecoder that does actual decoding work.
+  scoped_ptr<AudioDecoder> decoder_;
+
+  // The destination for the decoded buffers.
+  interfaces::AudioDecoderClientPtr client_;
+
+  DISALLOW_COPY_AND_ASSIGN(MojoAudioDecoderService);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_MOJO_SERVICES_MOJO_AUDIO_DECODER_SERVICE_H_