Adding an interface to allow extention of the audio player for CRD and iOS.

remoting/client/audio_consumer.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 REMOTING_CLIENT_AUDIO_CONSUMER_H_
+#define REMOTING_CLIENT_AUDIO_CONSUMER_H_
+
+#include <memory>
+
+#include "remoting/proto/audio.pb.h"

[Sergey Ulanov, 2016/06/09 20:23:45]

don't need this include. Forward-declare AudioPacket. See
https://www.chromium.org/developers/coding-style#TOC-Forward-declarations-vs....

+
+namespace remoting {
+
+class AudioConsumer {

[Sergey Ulanov, 2016/06/09 20:23:45]

Add a comment to make it clear what this interface is used for.

+ public:
+  AudioConsumer();
+  virtual ~AudioConsumer();

[Sergey Ulanov, 2016/06/09 20:23:45]

I think you need {} here and for the constructor.

+
+  virtual void AddAudioPacket(std::unique_ptr<AudioPacket> packet) = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(AudioConsumer);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_CLIENT_AUDIO_CONSUMER_H_