Simplified AudioOutputStream interface.

chrome/renderer/media/audio_renderer_impl.cc

 // Copyright (c) 2010 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 "chrome/renderer/media/audio_renderer_impl.h"
 
 #include <math.h>
 
 #include "chrome/common/render_messages_params.h"
 #include "chrome/renderer/audio_message_filter.h"
@@ skipping 237 matching lines @@
   if (stopped_)
     return;
 
   // Make sure we don't call create more than once.
   DCHECK_EQ(0, stream_id_);
   stream_id_ = filter_->AddDelegate(this);
   io_loop_->AddDestructionObserver(this);
 
   ViewHostMsg_Audio_CreateStream_Params params;
   params.params = audio_params;
-  params.packet_size = 0;
+
+  // Let the browser choose packet size.
+  params.params.samples_per_packet = 0;

[scherkus (not reviewing), 2010/11/09 02:28:31]

is this documented anywhere in params or IPC message?

[Sergey Ulanov, 2010/11/09 22:29:58]

Added comments about it in render_messages_params.h

 
   filter_->Send(new ViewHostMsg_CreateAudioStream(0, stream_id_, params,
                                                   false));
 }
 
 void AudioRendererImpl::PlayTask() {
   DCHECK(MessageLoop::current() == io_loop_);
 
   filter_->Send(new ViewHostMsg_PlayAudioStream(0, stream_id_));
 }
@@ skipping 81 matching lines @@
   DCHECK(MessageLoop::current() == io_loop_);
 
   // We treat the IO loop going away the same as stopping.
   AutoLock auto_lock(lock_);
   if (stopped_)
     return;
 
   stopped_ = true;
   DestroyTask();
 }