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

Unified Diff: media/filters/audio_clock.cc

Issue 1109183002: media: Add extra logging in AudioClock. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/audio_clock.cc
diff --git a/media/filters/audio_clock.cc b/media/filters/audio_clock.cc
index c561c27be0f2e8d71a4bcdcd341a0ac14f860313..c7f18ab7aef1db5197bb39e2e8e9578dcbdb3079 100644
--- a/media/filters/audio_clock.cc
+++ b/media/filters/audio_clock.cc
@@ -81,8 +81,10 @@ void AudioClock::WroteAudio(int frames_written,
}
base::TimeDelta AudioClock::TimeUntilPlayback(base::TimeDelta timestamp) const {
- DCHECK(timestamp >= front_timestamp_);
- DCHECK(timestamp <= back_timestamp_);
+ DCHECK(timestamp >= front_timestamp_) << ": " << timestamp << ", "
DaleCurtis 2015/04/28 18:28:12 I think you can just use DCHECK_GE() now?
xhwang 2015/04/28 19:51:14 Done.
+ << front_timestamp_;
+ DCHECK(timestamp <= back_timestamp_) << ": " << timestamp << ", "
+ << back_timestamp_;
int64_t frames_until_timestamp = 0;
double timestamp_us = timestamp.InMicroseconds();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698