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

Unified Diff: media/cast/transport/cast_transport_sender_impl.cc

Issue 196433002: Cast: Log sender side packet events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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/transport/cast_transport_sender_impl.cc
diff --git a/media/cast/transport/cast_transport_sender_impl.cc b/media/cast/transport/cast_transport_sender_impl.cc
index 7754e54e084792b2c03108b9d6f5bec745f36a2c..46dfaaf2d2aac8565819fb102c28f1bf93c6a351 100644
--- a/media/cast/transport/cast_transport_sender_impl.cc
+++ b/media/cast/transport/cast_transport_sender_impl.cc
@@ -55,11 +55,12 @@ CastTransportSenderImpl::CastTransportSenderImpl(
local_end_point,
remote_end_point,
status_callback)),
+ logging_(logging_config),
pacer_(clock,
+ &logging_,
external_transport ? external_transport : transport_.get(),
transport_task_runner),
rtcp_builder_(&pacer_),
- logging_(logging_config),
raw_events_callback_(raw_events_callback) {
if (!raw_events_callback_.is_null()) {
DCHECK(logging_config.enable_raw_data_collection);
@@ -80,8 +81,9 @@ CastTransportSenderImpl::~CastTransportSenderImpl() {
void CastTransportSenderImpl::InitializeAudio(
const CastTransportAudioConfig& config) {
+ pacer_.RegisterAudioSsrc(config.base.ssrc);
audio_sender_.reset(new TransportAudioSender(
- config, clock_, transport_task_runner_, &pacer_));
+ config, clock_, &logging_, transport_task_runner_, &pacer_));
if (audio_sender_->initialized())
status_callback_.Run(TRANSPORT_AUDIO_INITIALIZED);
else
@@ -90,8 +92,9 @@ void CastTransportSenderImpl::InitializeAudio(
void CastTransportSenderImpl::InitializeVideo(
const CastTransportVideoConfig& config) {
+ pacer_.RegisterVideoSsrc(config.base.ssrc);
video_sender_.reset(new TransportVideoSender(
- config, clock_, transport_task_runner_, &pacer_));
+ config, clock_, &logging_, transport_task_runner_, &pacer_));
if (video_sender_->initialized())
status_callback_.Run(TRANSPORT_VIDEO_INITIALIZED);
else

Powered by Google App Engine
This is Rietveld 408576698