Redesign the copresence audio handlers.

components/copresence/mediums/audio/audio_manager_stub.cc

Index: components/copresence/mediums/audio/audio_manager_stub.cc
diff --git a/components/copresence/mediums/audio/audio_manager_stub.cc b/components/copresence/mediums/audio/audio_manager_stub.cc
new file mode 100644
index 0000000000000000000000000000000000000000..65a89d3f2e0433b069cbdced47706c715df9ae00
--- /dev/null
+++ b/components/copresence/mediums/audio/audio_manager_stub.cc
@@ -0,0 +1,52 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[Charlie, 2014/10/22 01:03:29]

I think the typical pattern is that stub classes live in the *_unittest.cc file
that uses them. Then you don't need *_stub.h either. Is this used in multiple
places?

Ditto for the other stub files.

[Daniel Erat, 2014/10/22 16:34:35]

agreed that you can do this if these are only used in a single test file. if
not, you can probably just inline the implementation in the .h file and get rid
of this one (at least, i wouldn't care much about
http://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-in...
for a stub file...)

[rkc, 2014/10/22 18:21:47]

Sure.
Done.

[rkc, 2014/10/22 18:21:47]

Sure.
Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/copresence/mediums/audio/audio_manager_stub.h"
+
+#include "components/copresence/public/copresence_constants.h"
+
+namespace copresence {
+
+AudioManagerStub::AudioManagerStub() {
+}
+
+AudioManagerStub::~AudioManagerStub() {
+}
+
+void AudioManagerStub::Initialize(const DecodeSamplesCallback& decode_cb,
+                                  const EncodeTokenCallback& encode_cb) {
+}
+
+void AudioManagerStub::StartPlaying(AudioType type) {
+  playing_[type] = true;
+}
+
+void AudioManagerStub::StopPlaying(AudioType type) {
+  playing_[type] = false;
+}
+
+void AudioManagerStub::StartRecording(AudioType type) {
+  recording_[type] = true;
+}
+
+void AudioManagerStub::StopRecording(AudioType type) {
+  recording_[type] = false;
+}
+
+void AudioManagerStub::SetToken(AudioType type,
+                                const std::string& url_unsafe_token) {
+}
+const std::string& AudioManagerStub::GetToken(AudioType type) {
+  return token_;
+}
+
+bool AudioManagerStub::IsRecording(AudioType type) {
+  return recording_[type];
+}
+
+bool AudioManagerStub::IsPlaying(AudioType type) {
+  return playing_[type];
+}
+
+}  // namespace copresence