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

remoting/client/fake_audio_consumer.cc

Index: remoting/client/fake_audio_consumer.cc
diff --git a/remoting/client/fake_audio_consumer.cc b/remoting/client/fake_audio_consumer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a19df2c6d7dc1e5cf8147cfdecbef372cf3b8bd0
--- /dev/null
+++ b/remoting/client/fake_audio_consumer.cc
@@ -0,0 +1,20 @@
+// 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.
+
+#include "remoting/client/fake_audio_consumer.h"
+#include "remoting/proto/audio.pb.h"
+
+namespace remoting {
+
+FakeAudioConsumer::FakeAudioConsumer() : weak_factory_(this) {}

[Lambros, 2016/06/17 23:51:10]

You should maybe name this MockAudioConsumer, as it behaves like a mock. The
only implementation you need is to record how many times the method gets called.

+
+FakeAudioConsumer::~FakeAudioConsumer() {}
+
+void FakeAudioConsumer::AddAudioPacket(std::unique_ptr<AudioPacket> packet) {}

[Lambros, 2016/06/17 23:51:10]

You could increment a counter here, which would let you set expectations in
tests.

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

Ok sounds good, I will give this a try and see if I can make the test ... test
something :)

+
+base::WeakPtr<FakeAudioConsumer> FakeAudioConsumer::GetWeakPtr() {
+  return weak_factory_.GetWeakPtr();
+}
+
+}  // namespace remoting