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

Unified Diff: chrome/renderer/media/cast_transport_sender_ipc.cc

Issue 281453003: Cast: Simplify code path for RTCP sender report (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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: chrome/renderer/media/cast_transport_sender_ipc.cc
diff --git a/chrome/renderer/media/cast_transport_sender_ipc.cc b/chrome/renderer/media/cast_transport_sender_ipc.cc
index 1dd3906329418d62aea78c5818c5ae0393606033..cb5b741f2dc4ad9502bdba0bbbbc83e55a31d0e3 100644
--- a/chrome/renderer/media/cast_transport_sender_ipc.cc
+++ b/chrome/renderer/media/cast_transport_sender_ipc.cc
@@ -63,7 +63,9 @@ void CastTransportSenderIPC::InsertCodedVideoFrame(
void CastTransportSenderIPC::SendRtcpFromRtpSender(
uint32 packet_type_flags,
- const media::cast::transport::RtcpSenderInfo& sender_info,
+ uint32 ntp_seconds,
+ uint32 ntp_fraction,
+ uint32 rtp_timestamp,
const media::cast::transport::RtcpDlrrReportBlock& dlrr,
uint32 sending_ssrc,
const std::string& c_name) {
@@ -71,10 +73,12 @@ void CastTransportSenderIPC::SendRtcpFromRtpSender(
data.packet_type_flags = packet_type_flags;
data.sending_ssrc = sending_ssrc;
data.c_name = c_name;
+ data.ntp_seconds = ntp_seconds;
+ data.ntp_fraction = ntp_fraction;
+ data.rtp_timestamp = rtp_timestamp;
Send(new CastHostMsg_SendRtcpFromRtpSender(
channel_id_,
data,
- sender_info,
dlrr));
}
@@ -86,17 +90,6 @@ void CastTransportSenderIPC::ResendPackets(
missing_packets));
}
-void CastTransportSenderIPC::SubscribeAudioRtpStatsCallback(
- const media::cast::transport::CastTransportRtpStatistics& callback) {
- audio_rtp_callback_ = callback;
-}
-
-void CastTransportSenderIPC::SubscribeVideoRtpStatsCallback(
- const media::cast::transport::CastTransportRtpStatistics& callback) {
- video_rtp_callback_ = callback;
-}
-
-
void CastTransportSenderIPC::OnReceivedPacket(
const media::cast::Packet& packet) {
if (!packet_callback_.is_null()) {
@@ -114,16 +107,6 @@ void CastTransportSenderIPC::OnNotifyStatusChange(
status_callback_.Run(status);
}
-void CastTransportSenderIPC::OnRtpStatistics(
- bool audio,
- const media::cast::transport::RtcpSenderInfo& sender_info,
- base::TimeTicks time_sent,
- uint32 rtp_timestamp) {
- const media::cast::transport::CastTransportRtpStatistics& callback =
- audio ? audio_rtp_callback_ : video_rtp_callback_;
- callback.Run(sender_info, time_sent, rtp_timestamp);
-}
-
void CastTransportSenderIPC::OnRawEvents(
const std::vector<media::cast::PacketEvent>& packet_events) {
raw_events_callback_.Run(packet_events);

Powered by Google App Engine
This is Rietveld 408576698