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

Unified Diff: media/cast/test/utility/in_process_receiver.h

Issue 225023010: [Cast] Refactor/clean-up VideoReceiver to match AudioReceiver as closely as possible. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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/cast/test/utility/in_process_receiver.h
diff --git a/media/cast/test/utility/in_process_receiver.h b/media/cast/test/utility/in_process_receiver.h
index a5a2addd5c2e0bf2a82a7704be82c179f9552529..4867f3f3c27a66d439c155a7eb810e90bc9d7213 100644
--- a/media/cast/test/utility/in_process_receiver.h
+++ b/media/cast/test/utility/in_process_receiver.h
@@ -75,7 +75,7 @@ class InProcessReceiver {
virtual void OnAudioFrame(scoped_ptr<PcmAudioFrame> audio_frame,
const base::TimeTicks& playout_time) = 0;
virtual void OnVideoFrame(const scoped_refptr<VideoFrame>& video_frame,
- const base::TimeTicks& render_time) = 0;
+ const base::TimeTicks& playout_time) = 0;
// Helper method that creates |transport_| and |cast_receiver_|, starts
// |transport_| receiving, and requests the first audio/video frame.
@@ -98,7 +98,8 @@ class InProcessReceiver {
const base::TimeTicks& playout_time,
bool is_continuous);
void GotVideoFrame(const scoped_refptr<VideoFrame>& video_frame,
- const base::TimeTicks& render_time);
+ const base::TimeTicks& playout_time,
+ bool is_continuous);
hubbe 2014/04/07 18:40:17 Document |is_continuous| (and perhaps the entire m
miu 2014/04/08 00:59:41 Done. But, rather than duplicate comments, I've p
void PullNextAudioFrame();
void PullNextVideoFrame();

Powered by Google App Engine
This is Rietveld 408576698