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

Unified Diff: content/renderer/media/render_audiosourceprovider.cc

Issue 9655018: Make AudioParameters a class instead of a struct (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix copyright years Created 8 years, 9 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: content/renderer/media/render_audiosourceprovider.cc
diff --git a/content/renderer/media/render_audiosourceprovider.cc b/content/renderer/media/render_audiosourceprovider.cc
index 39bb84c7e7537b746cba636f7a73be7ad7c8cf3e..5159d4dff1ff87d607f2714a4ee3a30402843baa 100644
--- a/content/renderer/media/render_audiosourceprovider.cc
+++ b/content/renderer/media/render_audiosourceprovider.cc
@@ -14,7 +14,7 @@ using WebKit::WebVector;
RenderAudioSourceProvider::RenderAudioSourceProvider()
: is_initialized_(false),
channels_(0),
- sample_rate_(0.0),
+ sample_rate_(0),
is_running_(false),
volume_(1.0),
renderer_(NULL),
@@ -73,29 +73,22 @@ void RenderAudioSourceProvider::GetVolume(double* volume) {
}
void RenderAudioSourceProvider::Initialize(
- size_t buffer_size,
- int channels,
- double sample_rate,
- AudioParameters::Format latency_format,
- RenderCallback* renderer) {
+ const AudioParameters& params, RenderCallback* renderer) {
base::AutoLock auto_lock(sink_lock_);
CHECK(!is_initialized_);
renderer_ = renderer;
- default_sink_->Initialize(buffer_size,
- channels,
- sample_rate,
- latency_format,
- renderer);
+ default_sink_->Initialize(params, renderer);
+
+ // Keep track of the format in case the client hasn't yet been set.
+ channels_ = params.channels();
+ sample_rate_ = params.sample_rate();
if (client_) {
// Inform WebKit about the audio stream format.
- client_->setFormat(channels, sample_rate);
+ client_->setFormat(channels_, sample_rate_);
}
- // Keep track of the format in case the client hasn't yet been set.
- channels_ = channels;
- sample_rate_ = sample_rate;
is_initialized_ = true;
}
« no previous file with comments | « content/renderer/media/render_audiosourceprovider.h ('k') | content/renderer/media/renderer_webaudiodevice_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698