AudioConverter: Express delay in frames rather than msec.

media/audio/virtual_audio_output_stream.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/virtual_audio_output_stream.h"
 
 #include <stdint.h>
 
 #include "base/logging.h"
 #include "media/audio/virtual_audio_input_stream.h"
@@ skipping 56 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   volume_ = volume;
 }
 
 void VirtualAudioOutputStream::GetVolume(double* volume) {
   DCHECK(thread_checker_.CalledOnValidThread());
   *volume = volume_;
 }
 
 double VirtualAudioOutputStream::ProvideInput(AudioBus* audio_bus,
-                                              base::TimeDelta buffer_delay) {
+                                              uint32_t frames_delayed) {
   // Note: This method may be invoked on any one thread, depending on the
   // platform.
   DCHECK(callback_);
 
-  DCHECK_GE(buffer_delay, base::TimeDelta());
-  const int64_t upstream_delay_in_bytes = params_.GetBytesPerSecond() *
-                                          buffer_delay /
-                                          base::TimeDelta::FromSeconds(1);
+  DCHECK_GE(frames_delayed, 0U);
+  const int64_t upstream_delay_in_bytes =

[DaleCurtis, 2016/05/24 20:27:02]

Mixing of int64_t + uint32_t seems unusual?

[chcunningham, 2016/05/25 18:21:14]

Made int64_t -> uint32_t. Not really sure why it was done this way. Originally
from: 

https://codereview.chromium.org/756733003/diff/1/media/audio/virtual_audio_ou...

Will confirm with miu@ before I land.

+      std::round(params_.GetBytesPerFrame() * frames_delayed);
   const int frames = callback_->OnMoreData(
       audio_bus, static_cast<uint32_t>(upstream_delay_in_bytes), 0);
   if (frames < audio_bus->frames())
     audio_bus->ZeroFramesPartial(frames, audio_bus->frames() - frames);
 
   return frames > 0 ? volume_ : 0;
 }
 
 }  // namespace media