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

Unified Diff: media/filters/audio_renderer_impl.cc

Issue 16823003: Replace erroneous use of base::Time with base::TimeTicks throughout media code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: s/PresubmitPromptWarning/PresubmitPromptOrNotify/ Created 7 years, 6 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/filters/audio_renderer_impl.h ('k') | media/filters/audio_renderer_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/audio_renderer_impl.cc
diff --git a/media/filters/audio_renderer_impl.cc b/media/filters/audio_renderer_impl.cc
index 9afd550a658c67c03220e1028130c9b3801f4c57..8a4cd23eacba9c18dff08b873942af8fac8bf0a2 100644
--- a/media/filters/audio_renderer_impl.cc
+++ b/media/filters/audio_renderer_impl.cc
@@ -49,7 +49,7 @@ AudioRendererImpl::AudioRendererImpl(
sink_(sink),
decoder_selector_(new AudioDecoderSelector(
message_loop, decoders.Pass(), set_decryptor_ready_cb)),
- now_cb_(base::Bind(&base::Time::Now)),
+ now_cb_(base::Bind(&base::TimeTicks::Now)),
state_(kUninitialized),
sink_playing_(false),
pending_read_(false),
@@ -524,7 +524,7 @@ uint32 AudioRendererImpl::FillBuffer(uint8* dest,
// Otherwise the buffer has data we can send to the device.
frames_written = algorithm_->FillBuffer(dest, requested_frames);
if (frames_written == 0) {
- base::Time now = now_cb_.Run();
+ const base::TimeTicks now = now_cb_.Run();
if (received_end_of_stream_ && !rendered_end_of_stream_ &&
now >= earliest_end_time_) {
@@ -598,7 +598,8 @@ uint32 AudioRendererImpl::FillBuffer(uint8* dest,
}
void AudioRendererImpl::UpdateEarliestEndTime_Locked(
- int frames_filled, base::TimeDelta playback_delay, base::Time time_now) {
+ int frames_filled, const base::TimeDelta& playback_delay,
+ const base::TimeTicks& time_now) {
if (frames_filled <= 0)
return;
« no previous file with comments | « media/filters/audio_renderer_impl.h ('k') | media/filters/audio_renderer_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698