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

Unified Diff: media/filters/audio_renderer_impl.h

Issue 534073002: Switch to using media::TimeSource inside media::RendererImpl. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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/filters/audio_renderer_impl.h
diff --git a/media/filters/audio_renderer_impl.h b/media/filters/audio_renderer_impl.h
index 086cf99b116402069c77ecd82856492770841b68..1e79a5a26be5e915973648baaab1c7277d6da6bb 100644
--- a/media/filters/audio_renderer_impl.h
+++ b/media/filters/audio_renderer_impl.h
@@ -73,12 +73,12 @@ class MEDIA_EXPORT AudioRendererImpl
virtual void SetPlaybackRate(float rate) OVERRIDE;
virtual void SetMediaTime(base::TimeDelta time) OVERRIDE;
virtual base::TimeDelta CurrentMediaTime() OVERRIDE;
+ virtual base::TimeDelta ApproximateCurrentMediaTime() OVERRIDE;
// AudioRenderer implementation.
virtual void Initialize(DemuxerStream* stream,
const PipelineStatusCB& init_cb,
const StatisticsCB& statistics_cb,
- const TimeCB& time_cb,
const BufferingStateCB& buffering_state_cb,
const base::Closure& ended_cb,
const PipelineStatusCB& error_cb) OVERRIDE;
@@ -213,7 +213,6 @@ class MEDIA_EXPORT AudioRendererImpl
// Callbacks provided during Initialize().
PipelineStatusCB init_cb_;
- TimeCB time_cb_;
BufferingStateCB buffering_state_cb_;
base::Closure ended_cb_;
PipelineStatusCB error_cb_;
@@ -250,7 +249,7 @@ class MEDIA_EXPORT AudioRendererImpl
base::TimeDelta start_timestamp_;
base::TimeDelta ended_timestamp_;
- base::TimeDelta last_timestamp_update_;
+ base::TimeTicks last_render_ticks_;
xhwang 2014/09/08 22:56:06 As discussed offline, it would be nicer to add som
scherkus (not reviewing) 2014/09/09 18:17:18 Done.
// End variables which must be accessed under |lock_|. ----------------------

Powered by Google App Engine
This is Rietveld 408576698