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>
+
+namespace remoting {
+
+class AudioPacket;
+
+class AudioConsumer {

[Sergey Ulanov, 2016/06/18 00:43:38]

Please add a comment here to explain how this interface is used.

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

[Sergey Ulanov, 2016/06/18 00:43:38]

make the destructor protected - it's not part of the interface as you don't pass
AudioConsumer ownership anywhere.

[Sergey Ulanov, 2016/06/18 00:43:38]

don't need ;
Then run git cl format - it will add space between ) and {

[nicholss, 2016/06/20 17:47:51]

I think the mac formatter is not as good as the linux formatter. I had already
run the formatter as is, and without the ; and it remained as (){} so I will
just have to remember this style rule.

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