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

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: Account for faster input than configured max FPS. 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 9b6e62e22791c39f66213beb40b233fa08bf9612..ad0a6d88937cc81219f5853e7bf0a3d262ffc0ec 100644
--- a/media/cast/sender/video_sender.h
+++ b/media/cast/sender/video_sender.h
@@ -54,7 +54,6 @@ class VideoSender : public FrameSender,
const base::TimeTicks& capture_time);
protected:
- virtual int GetNumberOfFramesInEncoder() const OVERRIDE;
virtual void OnAck(uint32 frame_id) OVERRIDE;
private:
@@ -75,6 +74,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