Move VoiceEngineObserver into AudioSendStream so that we detect typing noises and return properly i…

webrtc/audio_state.h

+/*
+ *  Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+#ifndef WEBRTC_AUDIO_STATE_H_
+#define WEBRTC_AUDIO_STATE_H_
+
+namespace webrtc {
+
+class AudioDeviceModule;
+class VoiceEngine;
+
+// AudioState holds the state which must be shared between multiple instances of
+// webrtc::Call for audio processing purposes.
+class AudioState {
+ public:
+  struct Config {
+    // VoiceEngine used for audio streams and audio/video synchronization.

[tommi, 2015/10/29 16:28:50]

what about ownership/lifetime? (same for voice_engine)

[the sun, 2015/10/30 12:19:01]

Made it clear in the comment, or did you have anything else in mind?

[tommi, 2015/11/02 13:46:53]

Thanks, comment is what I meant.

[the sun, 2015/11/03 12:41:07]

Acknowledged.

+    VoiceEngine* voice_engine = nullptr;
+
+    // The AudioDeviceModule associated with the Calls.
+    AudioDeviceModule* audio_device_module = nullptr;
+  };
+
+  static AudioState* Create(const AudioState::Config& config);
+
+  virtual ~AudioState() {}
+};
+}  // namespace webrtc
+
+#endif  // WEBRTC_AUDIO_STATE_H_