Simplify interface for media/cast: CastTransportSenderImpl

media/cast/net/cast_transport_config.h

Index: media/cast/net/cast_transport_config.h
diff --git a/media/cast/net/cast_transport_config.h b/media/cast/net/cast_transport_config.h
index 82823ad3c5712303bcf39af1e5d3c886a5ed0600..1c10799477f42f74582ffcb5416f09b49dc5902a 100644
--- a/media/cast/net/cast_transport_config.h
+++ b/media/cast/net/cast_transport_config.h
@@ -138,6 +138,10 @@ class PacketSender {
   // Returns the number of bytes ever sent.
   virtual int64_t GetBytesSent() = 0;
 
+  virtual void StartReceiving(

[miu, 2016/02/22 22:38:48]

1. These methods don't belong in a "PacketSender" interface.  Looking at what
they're here for, perhaps this class should be renamed to something like, say,
PacketTransporter?

2. Please add comments for these methods.

[xjz, 2016/02/23 21:51:46]

Agree. Will rename it in a following CL together with the renaming of
CastTransportSender. Added a TODO comment here for now.
Done.

+      const PacketReceiverCallbackWithStatus& packet_receiver) = 0;
+  virtual void StopReceiving() = 0;
+
   virtual ~PacketSender() {}
 };