Express audio delay more precisely in frames rather than milliseconds.

media/audio/audio_output_stream_sink.cc

 // Copyright 2014 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/audio_output_stream_sink.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "media/audio/audio_manager.h"
@@ skipping 65 matching lines @@
 }
 
 int AudioOutputStreamSink::OnMoreData(AudioBus* dest,
                                       uint32_t total_bytes_delay,
                                       uint32_t frames_skipped) {
   // Note: Runs on the audio thread created by the OS.
   base::AutoLock al(callback_lock_);
   if (!active_render_callback_)
     return 0;
 
-  return active_render_callback_->Render(
-      dest, total_bytes_delay * 1000.0 / active_params_.GetBytesPerSecond(), 0);
+  uint32_t frames_delayed = std::round(static_cast<double>(total_bytes_delay) /
+                                       active_params_.GetBytesPerFrame());
+
+  return active_render_callback_->Render(dest, frames_delayed, 0);

[chcunningham, 2016/02/11 01:25:42]

frames_skipped is not propagated? Anyone know why?

[DaleCurtis, 2016/02/11 01:57:15]

This is only used for Mojo demos currently I think. Feel free to forward it.

[chcunningham, 2016/02/11 21:02:52]

Done.

[Henrik Grunell, 2016/02/12 17:47:01]

|frames_skipped| is only used for WebRTC. Other clients may possibly use it in
the future.

[chcunningham, 2016/02/17 01:56:52]

Acknowledged.

 }
 
 void AudioOutputStreamSink::OnError(AudioOutputStream* stream) {
   // Note: Runs on the audio thread created by the OS.
   base::AutoLock al(callback_lock_);
   if (active_render_callback_)
     active_render_callback_->OnRenderError();
 }
 
 void AudioOutputStreamSink::DoStart(const AudioParameters& params) {
@@ skipping 41 matching lines @@
   DCHECK(audio_task_runner_->BelongsToCurrentThread());
   stream_->SetVolume(volume);
 }
 
 void AudioOutputStreamSink::ClearCallback() {
   base::AutoLock al(callback_lock_);
   active_render_callback_ = NULL;
 }
 
 }  // namepace media