Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(653)

Unified Diff: media/audio/virtual_audio_output_stream.cc

Issue 2004283002: AudioConverter: Express delay in frames rather than msec. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Missed files & removed rounding Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/audio/virtual_audio_output_stream.h ('k') | media/base/audio_buffer_converter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/virtual_audio_output_stream.cc
diff --git a/media/audio/virtual_audio_output_stream.cc b/media/audio/virtual_audio_output_stream.cc
index 1740b405d17c36069271113b534453b05f8f51f1..9017ec3c157cabedf7503becfcc209d2b4e08971 100644
--- a/media/audio/virtual_audio_output_stream.cc
+++ b/media/audio/virtual_audio_output_stream.cc
@@ -74,17 +74,15 @@ void VirtualAudioOutputStream::GetVolume(double* 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);
- const int frames = callback_->OnMoreData(
- audio_bus, static_cast<uint32_t>(upstream_delay_in_bytes), 0);
+ const uint32_t upstream_delay_in_bytes =
+ params_.GetBytesPerFrame() * frames_delayed;
+ const int frames =
+ callback_->OnMoreData(audio_bus, upstream_delay_in_bytes, 0);
if (frames < audio_bus->frames())
audio_bus->ZeroFramesPartial(frames, audio_bus->frames() - frames);
« no previous file with comments | « media/audio/virtual_audio_output_stream.h ('k') | media/base/audio_buffer_converter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698