Make AudioParameters a class instead of a struct

content/renderer/pepper/pepper_platform_audio_output_impl.cc

 // 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.
 
 #include "content/renderer/pepper/pepper_platform_audio_output_impl.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop_proxy.h"
 #include "build/build_config.h"
@@ skipping 11 matching lines @@
 
 PepperPlatformAudioOutputImpl::~PepperPlatformAudioOutputImpl() {
   // Make sure we have been shut down. Warning: this will usually happen on
   // the I/O thread!
   DCHECK_EQ(0, stream_id_);
   DCHECK(!client_);
 }
 
 // static
 PepperPlatformAudioOutputImpl* PepperPlatformAudioOutputImpl::Create(
-    uint32_t sample_rate,
-    uint32_t sample_count,
+    int sample_rate,
+    int frames_per_buffer,
     webkit::ppapi::PluginDelegate::PlatformAudioCommonClient* client) {
   scoped_refptr<PepperPlatformAudioOutputImpl> audio_output(
       new PepperPlatformAudioOutputImpl);
-  if (audio_output->Initialize(sample_rate, sample_count, client)) {
+  if (audio_output->Initialize(sample_rate, frames_per_buffer, client)) {
     // Balanced by Release invoked in
     // PepperPlatformAudioOutputImpl::ShutDownOnIOThread().
     return audio_output.release();
   }
   return NULL;
 }
 
 bool PepperPlatformAudioOutputImpl::StartPlayback() {
   if (filter_) {
     ChildProcess::current()->io_message_loop()->PostTask(
@@ skipping 19 matching lines @@
 void PepperPlatformAudioOutputImpl::ShutDown() {
   // Called on the main thread to stop all audio callbacks. We must only change
   // the client on the main thread, and the delegates from the I/O thread.
   client_ = NULL;
   ChildProcess::current()->io_message_loop()->PostTask(
       FROM_HERE,
       base::Bind(&PepperPlatformAudioOutputImpl::ShutDownOnIOThread, this));
 }
 
 bool PepperPlatformAudioOutputImpl::Initialize(
-    uint32_t sample_rate,
-    uint32_t sample_count,
+    int sample_rate,
+    int frames_per_buffer,
     webkit::ppapi::PluginDelegate::PlatformAudioCommonClient* client) {
   DCHECK(client);
   // Make sure we don't call init more than once.
   DCHECK_EQ(0, stream_id_);
 
   client_ = client;
 
-  AudioParameters params;
-  const uint32_t kMaxSampleCountForLowLatency = 2048;
+  AudioParameters::Format format;
+  const int kMaxFramesForLowLatency = 2048;
   // Use the low latency back end if the client request is compatible, and
   // the sample count is low enough to justify using AUDIO_PCM_LOW_LATENCY.
   if (sample_rate == audio_hardware::GetOutputSampleRate() &&
-      sample_count <= kMaxSampleCountForLowLatency &&
-      sample_count % audio_hardware::GetOutputBufferSize() == 0)
-    params.format = AudioParameters::AUDIO_PCM_LOW_LATENCY;
-  else
-    params.format = AudioParameters::AUDIO_PCM_LINEAR;
-  params.channels = 2;
-  params.sample_rate = sample_rate;
-  params.bits_per_sample = 16;
-  params.samples_per_packet = sample_count;
+      frames_per_buffer <= kMaxFramesForLowLatency &&
+      frames_per_buffer % audio_hardware::GetOutputBufferSize() == 0) {
+    format = AudioParameters::AUDIO_PCM_LOW_LATENCY;
+  } else {
+    format = AudioParameters::AUDIO_PCM_LINEAR;
+  }
+
+  AudioParameters params(format, CHANNEL_LAYOUT_STEREO, sample_rate,
+                         16, frames_per_buffer);
 
   ChildProcess::current()->io_message_loop()->PostTask(
       FROM_HERE,
       base::Bind(&PepperPlatformAudioOutputImpl::InitializeOnIOThread,
                  this, params));
   return true;
 }
 
 void PepperPlatformAudioOutputImpl::InitializeOnIOThread(
     const AudioParameters& params) {
@@ skipping 44 matching lines @@
     // Must dereference the client only on the main thread. Shutdown may have
     // occurred while the request was in-flight, so we need to NULL check.
     if (client_)
       client_->StreamCreated(handle, length, socket_handle);
   } else {
     main_message_loop_proxy_->PostTask(FROM_HERE,
         base::Bind(&PepperPlatformAudioOutputImpl::OnStreamCreated, this,
                    handle, socket_handle, length));
   }
 }