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

Unified Diff: media/cast/cast_config.h

Issue 562653004: Cast: First stab at implementing adaptive latency (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: minor changes after more manual testing 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/cast_config.h
diff --git a/media/cast/cast_config.h b/media/cast/cast_config.h
index 3a27702dd5cb6362dc4f9375894b20178f4f2e81..62ad9c23534d243be59d59d9321a38d7b95bee38 100644
--- a/media/cast/cast_config.h
+++ b/media/cast/cast_config.h
@@ -38,12 +38,13 @@ struct AudioSenderConfig {
int rtcp_interval;
// The total amount of time between a frame's capture/recording on the sender
- // and its playback on the receiver (i.e., shown to a user). This is fixed as
- // a value large enough to give the system sufficient time to encode,
+ // and its playback on the receiver (i.e., shown to a user). This should be
+ // set to a value large enough to give the system sufficient time to encode,
// transmit/retransmit, receive, decode, and render; given its run-time
// environment (sender/receiver hardware performance, network conditions,
// etc.).
- base::TimeDelta target_playout_delay;
+ base::TimeDelta min_playout_delay;
+ base::TimeDelta max_playout_delay;
// RTP payload type enum: Specifies the type/encoding of frame data.
int rtp_payload_type;
@@ -74,12 +75,13 @@ struct VideoSenderConfig {
int rtcp_interval;
// The total amount of time between a frame's capture/recording on the sender
- // and its playback on the receiver (i.e., shown to a user). This is fixed as
- // a value large enough to give the system sufficient time to encode,
+ // and its playback on the receiver (i.e., shown to a user). This should be
+ // set to a value large enough to give the system sufficient time to encode,
// transmit/retransmit, receive, decode, and render; given its run-time
// environment (sender/receiver hardware performance, network conditions,
// etc.).
- base::TimeDelta target_playout_delay;
+ base::TimeDelta min_playout_delay;
+ base::TimeDelta max_playout_delay;
// RTP payload type enum: Specifies the type/encoding of frame data.
int rtp_payload_type;

Powered by Google App Engine
This is Rietveld 408576698