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

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

Issue 9655018: Make AudioParameters a class instead of a struct (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests 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/audio_device_thread.cc
diff --git a/content/renderer/media/audio_device_thread.cc b/content/renderer/media/audio_device_thread.cc
index c76f999d817ee207e36ea517e9435ea1390f5b39..1575db0bdcd12fe3d702a553ed342182c2a10416 100644
--- a/content/renderer/media/audio_device_thread.cc
+++ b/content/renderer/media/audio_device_thread.cc
@@ -174,9 +174,9 @@ AudioDeviceThread::Callback::Callback(const AudioParameters& audio_parameters,
base::SharedMemoryHandle memory,
int memory_length)
: audio_parameters_(audio_parameters),
- samples_per_ms_(audio_parameters.sample_rate / 1000),
- bytes_per_ms_(audio_parameters.channels *
- (audio_parameters_.bits_per_sample / 8) *
+ samples_per_ms_(audio_parameters.samples_per_second() / 1000),
+ bytes_per_ms_(audio_parameters.channels() *
+ (audio_parameters_.bits_per_sample() / 8) *
samples_per_ms_),
shared_memory_(memory, false),
memory_length_(memory_length) {
@@ -193,10 +193,9 @@ void AudioDeviceThread::Callback::InitializeOnAudioThread() {
MapSharedMemory();
DCHECK(shared_memory_.memory() != NULL);
- audio_data_.reserve(audio_parameters_.channels);
- for (int i = 0; i < audio_parameters_.channels; ++i) {
- float* channel_data = new float[audio_parameters_.samples_per_packet];
+ audio_data_.reserve(audio_parameters_.channels());
+ for (int i = 0; i < audio_parameters_.channels(); ++i) {
+ float* channel_data = new float[audio_parameters_.samples_per_packet()];
audio_data_.push_back(channel_data);
}
}
-

Powered by Google App Engine
This is Rietveld 408576698