Cast: IPC from browser to renderer to send packet events from transport to cast library.

chrome/browser/media/cast_transport_host_filter.cc

Index: chrome/browser/media/cast_transport_host_filter.cc
diff --git a/chrome/browser/media/cast_transport_host_filter.cc b/chrome/browser/media/cast_transport_host_filter.cc
index fe5ccacd41a32078a547185cc1627122ad3ff3ae..35db8be70025b79ea06d641318f0d2e20978bfec 100644
--- a/chrome/browser/media/cast_transport_host_filter.cc
+++ b/chrome/browser/media/cast_transport_host_filter.cc
@@ -57,6 +57,13 @@ void CastTransportHostFilter::ReceivedRtpStatistics(
                                  rtp_timestamp));
 }
 
+void CastTransportHostFilter::RawEvents(
+    int32 channel_id,
+    const std::vector<media::cast::PacketEvent>& packet_events) {
+  if (!packet_events.empty())
+    Send(new CastMsg_RawEvents(channel_id, packet_events));
+}
+
 void CastTransportHostFilter::OnNew(
     int32 channel_id,
     const media::cast::transport::CastTransportConfig& config) {
@@ -66,13 +73,20 @@ void CastTransportHostFilter::OnNew(
     id_map_.Remove(channel_id);
   }
 
-  sender = media::cast::transport::CastTransportSender::
-      CreateCastTransportSender(
+  media::cast::CastLoggingConfig logging_config;
+  logging_config.enable_raw_data_collection = config.enable_raw_event_logging;

[hubbe, 2014/02/25 22:22:16]

This seems really weird to me.
What is the point of CreateCastTransportSender taking a "logging_config" and a
"config", if the "config" already contains the   only value that we set in the
logging_config??

[imcheng, 2014/02/25 23:14:56]

I was hoping to have a LoggingImpl in CastTransportSenderImpl without using a
scoped_ptr. I made some changes to CastLoggingConfig so that I don't have to
pass it in.

+
+  sender =
+      media::cast::transport::CastTransportSender::CreateCastTransportSender(
           &clock_,
           config,
+          logging_config,
           base::Bind(&CastTransportHostFilter::NotifyStatusChange,
                      base::Unretained(this),
                      channel_id),
+          base::Bind(&CastTransportHostFilter::RawEvents,
+                     base::Unretained(this),
+                     channel_id),
           base::MessageLoopProxy::current());
   sender->SetPacketReceiver(
           base::Bind(&CastTransportHostFilter::ReceivedPacket,