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

Unified Diff: media/cast/rtcp/rtcp.cc

Issue 192843002: Cast:Adding signaling and infrastructure for adjustable delay (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updates 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/rtcp/rtcp.cc
diff --git a/media/cast/rtcp/rtcp.cc b/media/cast/rtcp/rtcp.cc
index 958e47209159b923e7a462122f5bc5ba8d276664..1bf2edd726206f5b8d1a23ea594d9ba4931940f5 100644
--- a/media/cast/rtcp/rtcp.cc
+++ b/media/cast/rtcp/rtcp.cc
@@ -241,18 +241,18 @@ void Rtcp::SendRtcpFromRtpReceiver(
// Attach our NTP to all RTCP packets; with this information a "smart" sender
// can make decisions based on how old the RTCP message is.
- packet_type_flags |= RtcpSender::kRtcpRrtr;
+ packet_type_flags |= transport::kRtcpRrtr;
ConvertTimeTicksToNtp(now, &rrtr.ntp_seconds, &rrtr.ntp_fraction);
SaveLastSentNtpTime(now, rrtr.ntp_seconds, rrtr.ntp_fraction);
if (cast_message) {
- packet_type_flags |= RtcpSender::kRtcpCast;
+ packet_type_flags |= transport::kRtcpCast;
}
if (event_subscriber) {
- packet_type_flags |= RtcpSender::kRtcpReceiverLog;
+ packet_type_flags |= transport::kRtcpReceiverLog;
}
if (rtcp_mode_ == kRtcpCompound || now >= next_time_to_send_rtcp_) {
- packet_type_flags |= RtcpSender::kRtcpRr;
+ packet_type_flags |= transport::kRtcpRr;
report_block.remote_ssrc = 0; // Not needed to set send side.
report_block.media_ssrc = remote_ssrc_; // SSRC of the RTP packet sender.
@@ -281,17 +281,18 @@ void Rtcp::SendRtcpFromRtpReceiver(
UpdateNextTimeToSendRtcp();
}
rtcp_sender_->SendRtcpFromRtpReceiver(
- packet_type_flags, &report_block, &rrtr, cast_message, event_subscriber);
+ packet_type_flags, &report_block, &rrtr, cast_message, event_subscriber,
+ target_delay_ms_);
}
void Rtcp::SendRtcpFromRtpSender(
const transport::RtcpSenderLogMessage& sender_log_message) {
DCHECK(transport_sender_);
- uint32 packet_type_flags = RtcpSender::kRtcpSr;
+ uint32 packet_type_flags = transport::kRtcpSr;
base::TimeTicks now = cast_environment_->Clock()->NowTicks();
if (sender_log_message.size()) {
- packet_type_flags |= RtcpSender::kRtcpSenderLog;
+ packet_type_flags |= transport::kRtcpSenderLog;
}
transport::RtcpSenderInfo sender_info;
@@ -304,7 +305,7 @@ void Rtcp::SendRtcpFromRtpSender(
transport::RtcpDlrrReportBlock dlrr;
if (!time_last_report_received_.is_null()) {
- packet_type_flags |= RtcpSender::kRtcpDlrr;
+ packet_type_flags |= transport::kRtcpDlrr;
dlrr.last_rr = last_report_received_;
uint32 delay_seconds = 0;
uint32 delay_fraction = 0;
@@ -392,6 +393,10 @@ void Rtcp::SetCastReceiverEventHistorySize(size_t size) {
rtcp_receiver_->SetCastReceiverEventHistorySize(size);
}
+void Rtcp::SetTargetDelay(base::TimeDelta target_delay) {
+ target_delay_ms_ = static_cast<uint16>(target_delay.InMilliseconds());
pwestin(chromium) 2014/03/11 22:07:55 Sanity check for large and negative values
mikhal1 2014/03/12 18:01:59 Done.
+}
+
void Rtcp::OnReceivedDelaySinceLastReport(uint32 receivers_ssrc,
uint32 last_report,
uint32 delay_since_last_report) {

Powered by Google App Engine
This is Rietveld 408576698