Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1021)

Unified Diff: media/base/multi_channel_resampler.h

Issue 10823175: Switch AudioRenderSink::Callback to use AudioBus. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Gotta catch'em all! Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/multi_channel_resampler.h
diff --git a/media/base/multi_channel_resampler.h b/media/base/multi_channel_resampler.h
index db95bc52776273168fb158020507885c47877556..a45f4f73bccd3448b81471292563de846583016d 100644
--- a/media/base/multi_channel_resampler.h
+++ b/media/base/multi_channel_resampler.h
@@ -5,24 +5,22 @@
#ifndef MEDIA_BASE_MULTI_CHANNEL_RESAMPLER_H_
#define MEDIA_BASE_MULTI_CHANNEL_RESAMPLER_H_
-#include <vector>
-
#include "base/callback.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/scoped_vector.h"
#include "media/base/sinc_resampler.h"
namespace media {
+class AudioBus;
// MultiChannelResampler is a multi channel wrapper for SincResampler; allowing
// high quality sample rate conversion of multiple channels at once.
class MEDIA_EXPORT MultiChannelResampler {
public:
- // Callback type for providing more data into the resampler. Expects |frames|
- // of data for all channels to be rendered into |destination|; zero padded if
- // not enough frames are available to satisfy the request.
- typedef base::Callback<void(const std::vector<float*>& destination,
- int frames)> ReadCB;
+ // Callback type for providing more data into the resampler. Expects AudioBus
+ // to be filled with |frames| of data upon return; zero padded if not enough
+ // frames are available to satisfy the request.
+ typedef base::Callback<void(AudioBus* audio_bus, int frames)> ReadCB;
// Constructs a MultiChannelResampler with the specified |read_cb|, which is
// used to acquire audio data for resampling. |io_sample_rate_ratio| is the
@@ -31,8 +29,8 @@ class MEDIA_EXPORT MultiChannelResampler {
const ReadCB& read_cb);
virtual ~MultiChannelResampler();
- // Resample |frames| of data from |read_cb_| into |destination|.
- void Resample(const std::vector<float*>& destination, int frames);
+ // Resamples |frames| of data from |read_cb_| into AudioBus.
+ void Resample(AudioBus* audio_bus, int frames);
private:
// SincResampler::ReadCB implementation. ProvideInput() will be called for
@@ -43,18 +41,14 @@ class MEDIA_EXPORT MultiChannelResampler {
// frames for every channel.
int last_frame_count_;
- // Sanity check to ensure |resampler_audio_data_| is properly allocated.
- int first_frame_count_;
-
// Source of data for resampling.
ReadCB read_cb_;
// Each channel has its own high quality resampler.
ScopedVector<SincResampler> resamplers_;
- // Buffer for audio data going into SincResampler from ReadCB. Owned by this
- // class and only temporarily passed out to ReadCB when data is required.
- std::vector<float*> resampler_audio_data_;
+ // Buffer for audio data going into SincResampler from ReadCB.
+ scoped_ptr<AudioBus> resampler_audio_bus_;
};
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698