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

Unified Diff: media/cast/sender/video_sender.h

Issue 560223002: [Cast] Limit frames in flight by duration, and not by number of frames. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed auto-eviction from PacketStorage, since that should never happen. 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/cast/sender/video_sender.h
diff --git a/media/cast/sender/video_sender.h b/media/cast/sender/video_sender.h
index 2c1ca5dfd1949d166472972976de946f8c874c7b..e7658abeea45433bcee1947022e34d87c92e459c 100644
--- a/media/cast/sender/video_sender.h
+++ b/media/cast/sender/video_sender.h
@@ -58,6 +58,7 @@ class VideoSender : public FrameSender,
protected:
virtual int GetNumberOfFramesInEncoder() const OVERRIDE;
+ virtual base::TimeDelta GetInFlightMediaDuration() const OVERRIDE;
virtual void OnAck(uint32 frame_id) OVERRIDE;
private:
@@ -78,6 +79,12 @@ class VideoSender : public FrameSender,
// The number of frames queued for encoding, but not yet sent.
int frames_in_encoder_;
+ // The duration of video queued for encoding, but not yet sent.
+ base::TimeDelta duration_in_encoder_;
+
+ // The timestamp of the frame that was last enqueued in |video_encoder_|.
+ base::TimeTicks last_enqueued_frame_reference_time_;
+
// Remember what we set the bitrate to before, no need to set it again if
// we get the same value.
uint32 last_bitrate_;

Powered by Google App Engine
This is Rietveld 408576698