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

Unified Diff: media/base/audio_converter.cc

Issue 1954633002: MEDIA_LOG for large encoded timestamp gaps in decoder stream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: New AudioTimestampValidator class and tests 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
Index: media/base/audio_converter.cc
diff --git a/media/base/audio_converter.cc b/media/base/audio_converter.cc
index 48686046fd71178ef38416a153c24641c95d96ff..be8f0752a0a0677285babc75f626c571f0fc7d58 100644
--- a/media/base/audio_converter.cc
+++ b/media/base/audio_converter.cc
@@ -190,13 +190,22 @@ void AudioConverter::SourceCallback(int fifo_frame_delay, AudioBus* dest) {
// Calculate the buffer delay for this callback.
base::TimeDelta buffer_delay = initial_delay_;
+ LOG(ERROR) << "initial delay:" << initial_delay_.InMicroseconds();
if (resampler_) {
- buffer_delay += base::TimeDelta::FromMicroseconds(
+ base::TimeDelta resampler_delay = base::TimeDelta::FromMicroseconds(
resampler_frame_delay_ * output_frame_duration_.InMicroseconds());
+ LOG(ERROR) << "resampler delay:" << resampler_delay.InMicroseconds()
+ << " frames:" << resampler_frame_delay_
+ << " output_dur:" << output_frame_duration_.InMicroseconds();
+ buffer_delay += resampler_delay;
}
if (audio_fifo_) {
- buffer_delay += base::TimeDelta::FromMicroseconds(
+ base::TimeDelta fifo_delay = base::TimeDelta::FromMicroseconds(
fifo_frame_delay * input_frame_duration_.InMicroseconds());
+ LOG(ERROR) << "fifo delay:" << fifo_delay.InMicroseconds()
+ << " frames:" << fifo_frame_delay
+ << " input_dur:" << input_frame_duration_.InMicroseconds();
+ buffer_delay += fifo_delay;
}
// If we only have a single input, avoid an extra copy.

Powered by Google App Engine
This is Rietveld 408576698