Avoids irregular OnMoreData callbacks on Windows using Core Audio

media/audio/win/audio_unified_win.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 "media/audio/win/audio_unified_win.h"
 
 #include <Functiondiscoverykeys_devpkey.h>
 
 #include "base/debug/trace_event.h"
 #include "base/time.h"
@@ skipping 228 matching lines @@
   }
 
   // Start input streaming data between the endpoint buffer and the audio
   // engine.
   HRESULT hr = audio_input_client_->Start();
   if (FAILED(hr)) {
     StopAndJoinThread(hr);
     return;
   }
 
-  // Reset the counter for number of rendered frames taking into account the
-  // fact that we always initialize the render side with silence.
-  UINT32 num_queued_frames = 0;
-  audio_output_client_->GetCurrentPadding(&num_queued_frames);
-  DCHECK_EQ(num_queued_frames, endpoint_render_buffer_size_frames_);
-  num_written_frames_ = num_queued_frames;
+  // Ensure that the endpoint buffer is prepared with silence.
+  UINT32 num_filled_frames = 0;
+  if (share_mode_ == AUDCLNT_SHAREMODE_SHARED) {
+    if (!FillRenderEndpointBufferWithSilence(&num_filled_frames)) {
+      DLOG(WARNING) << "Failed to prepare endpoint buffers with silence.";
+      return;
+    }
+    DCHECK_EQ(num_filled_frames, endpoint_render_buffer_size_frames_);
+  }
+  num_written_frames_ = num_filled_frames;
 
   // Start output streaming data between the endpoint buffer and the audio
   // engine.
   hr = audio_output_client_->Start();
   if (FAILED(hr)) {
     StopAndJoinThread(hr);
     return;
   }
 }
 
@@ skipping 294 matching lines @@
 
 void WASAPIUnifiedStream::StopAndJoinThread(HRESULT err) {
   CHECK(GetCurrentThreadId() == creating_thread_id_);
   DCHECK(audio_io_thread_.get());
   SetEvent(stop_streaming_event_.Get());
   audio_io_thread_->Join();
   audio_io_thread_.reset();
   HandleError(err);
 }
 
+bool WASAPIUnifiedStream::FillRenderEndpointBufferWithSilence(

[DaleCurtis, 2013/01/31 02:34:33]

Should this just be in CoreAudioUtil?

[henrika (OOO until Aug 14), 2013/01/31 14:29:38]

Good point. I will fix that.

+  UINT32* num_written_frames) {

[tommi (sloooow) - chröme, 2013/01/31 13:42:08]

indent

[henrika (OOO until Aug 14), 2013/01/31 14:29:38]

Done.

+  UINT32 num_queued_frames = 0;
+  HRESULT hr = audio_output_client_->GetCurrentPadding(&num_queued_frames);
+  if (FAILED(hr))
+    return false;
+
+  BYTE* data = NULL;
+  int num_frames_to_fill =
+      endpoint_render_buffer_size_frames_ - num_queued_frames;
+  hr = audio_render_client_->GetBuffer(num_frames_to_fill, &data);
+  if (FAILED(hr))
+    return false;
+
+  // Using the AUDCLNT_BUFFERFLAGS_SILENT flag eliminates the need to
+  // explicitly write silence data to the rendering buffer.
+  DVLOG(2) << "filling up " << num_frames_to_fill << " frames with silence";
+  hr = audio_render_client_->ReleaseBuffer(num_frames_to_fill,
+                                          AUDCLNT_BUFFERFLAGS_SILENT);

[tommi (sloooow) - chröme, 2013/01/31 13:42:08]

indent

[henrika (OOO until Aug 14), 2013/01/31 14:29:38]

Done.

+  if (FAILED(hr))
+    return false;
+
+  // Get the amount of valid, unread data that the endpoint buffer
+  // currently contains. This amount corresponds to the number of written
+  // audio frames.
+  hr = audio_output_client_->GetCurrentPadding(&num_queued_frames);
+  if (FAILED(hr))
+    return false;
+  *num_written_frames = num_queued_frames;
+
+  return SUCCEEDED(hr);

[tommi (sloooow) - chröme, 2013/01/31 13:42:08]

nit: you've already checked hr, so you can return true here.

[henrika (OOO until Aug 14), 2013/01/31 14:29:38]

I use the "hängslen och livrem" principle here ;-)

+}
+
 }  // namespace media