Simplify interface for media/cast: CastTransportSenderImpl

chrome/browser/media/cast_transport_host_filter.h

Index: chrome/browser/media/cast_transport_host_filter.h
diff --git a/chrome/browser/media/cast_transport_host_filter.h b/chrome/browser/media/cast_transport_host_filter.h
index 094d7e7f62c38b805752e1baf62472be06142a42..c9ec66c2a0ba688e9d43f94600ee94e1d04f59cb 100644
--- a/chrome/browser/media/cast_transport_host_filter.h
+++ b/chrome/browser/media/cast_transport_host_filter.h
@@ -20,29 +20,30 @@ class PowerSaveBlocker;
 
 namespace cast {
 
-class CastTransportHostFilter : public content::BrowserMessageFilter {
+class CastTransportHostFilter
+    : public content::BrowserMessageFilter,
+      media::cast::CastTransportSender::CastTransportClient {
  public:
   CastTransportHostFilter();
 
  private:
   ~CastTransportHostFilter() override;
 
-  void NotifyStatusChange(
-      int32 channel_id,
-      media::cast::CastTransportStatus result);
-  void SendRawEvents(
-      int32 channel_id,
-      scoped_ptr<std::vector<media::cast::FrameEvent>> frame_events,
-      scoped_ptr<std::vector<media::cast::PacketEvent>> packet_events);
   void SendRtt(int32 channel_id, uint32 ssrc, base::TimeDelta rtt);
   void SendCastMessage(int32 channel_id,
                        uint32 ssrc,
                        const media::cast::RtcpCastMessage& cast_message);
-  void ReceivedPacket(int32 channel_id, scoped_ptr<media::cast::Packet> packet);
 
   // BrowserMessageFilter implementation.
   bool OnMessageReceived(const IPC::Message& message) override;
 
+  // Override CastTransportSender::CastTransportClient.

[Irfan, 2015/12/16 19:44:34]

ha, my suggestion of "CastTransportEndPoint" was for CastTransportSender.

I am ok if you want to retain CastTransportClient here

[xjz, 2015/12/16 21:58:01]

The rename of "CastTransportSender" might in a later CL as Yuri suggested. I am
OK with either "Client" or "CastTransportClient". Rename it back to "Client" to
make it simpler for now :).

+  void OnStatusChange(media::cast::CastTransportStatus status) final;
+  void OnReceivedLoggingEvents(
+      scoped_ptr<std::vector<media::cast::FrameEvent>> frame_events,
+      scoped_ptr<std::vector<media::cast::PacketEvent>> packet_events) final;
+  void OnReceivedPackets(scoped_ptr<media::cast::Packet> packet) final;
+
   // Forwarding functions.
   void OnInitializeAudio(
       int32 channel_id,
@@ -85,6 +86,9 @@ class CastTransportHostFilter : public content::BrowserMessageFilter {
   // remoting content.
   scoped_ptr<content::PowerSaveBlocker> power_save_blocker_;
 
+  // Active channel id.
+  int32 channel_id_;
+
   base::WeakPtrFactory<CastTransportHostFilter> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CastTransportHostFilter);