Piping for audio encoding.

remoting/base/audio_encoder.h

Index: remoting/base/audio_encoder.h
diff --git a/remoting/base/audio_encoder.h b/remoting/base/audio_encoder.h
new file mode 100644
index 0000000000000000000000000000000000000000..da5d85f0b13015b6678967bec7c936ce2daef62f
--- /dev/null
+++ b/remoting/base/audio_encoder.h
@@ -0,0 +1,29 @@
+// Copyright (c) 2012 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_BASE_AUDIO_ENCODER_H_
+#define REMOTING_BASE_AUDIO_ENCODER_H_

[Sergey Ulanov, 2012/07/31 20:00:18]

can you please create new directory remoting/codec and put the new codec-related
code there instead of remoting/base. We will move video codecs there later.

[kxing, 2012/07/31 22:11:44]

Done.

+
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace remoting {
+
+class AudioCaptureData;
+class AudioPacket;
+
+class AudioEncoder {
+ public:
+  typedef base::Callback<void(scoped_ptr<AudioPacket>)> PacketEncodedCallback;
+
+  virtual ~AudioEncoder() {}
+
+  virtual void Encode(
+      scoped_ptr<AudioCaptureData> audio_capture_data,
+      const PacketEncodedCallback& packet_captured_callback) = 0;

[Sergey Ulanov, 2012/07/31 20:00:18]

Don't need this to be asyncronous. This function should just return
scoped_ptr<AudioPacket>, don't need the callback.
Ou video encode it also synchronous, even though the interface make it look like
it is asynchronous.

[kxing, 2012/07/31 22:11:44]

Done.

+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_BASE_AUDIO_ENCODER_H_