Remove voe::Channel::StopReceive() and associated logic.

webrtc/voice_engine/include/voe_base.h

Index: webrtc/voice_engine/include/voe_base.h
diff --git a/webrtc/voice_engine/include/voe_base.h b/webrtc/voice_engine/include/voe_base.h
index 751394e8d7bd023a967e34be064b5aa2b3522520..bdc1a2608ede77fadd13a1838aa6395a4d2c5577 100644
--- a/webrtc/voice_engine/include/voe_base.h
+++ b/webrtc/voice_engine/include/voe_base.h
@@ -165,7 +165,7 @@ class WEBRTC_DLLEXPORT VoEBase {
   virtual int StartReceive(int channel) = 0;

[ossu, 2016/10/27 09:40:56]

If you're removing StopReceive, shouldn't StartReceive go as well? Seems strange
that you can do one thing and not the other.
Also looks like StartReceive is not being called anywhere.

[the sun, 2016/10/27 10:44:02]

The reason I decided to keep it is that StartReceive() resets the discarded
packet count, which can be retrieved using the VoERTP_RTCP::GetRTPStatistics()
API. StopReceive() didn't have any function from what I could see.

[ossu, 2016/10/27 12:58:37]

Then I think you should rename it to something like RestartReceive or
ResetDiscardedPacketCount.

 
   // Stops receiving incoming RTP/RTCP packets on the specified |channel|.
-  virtual int StopReceive(int channel) = 0;
+  virtual int StopReceive(int channel)  { return 0; }
 
   // Starts forwarding the packets to the mixer/soundcard for a
   // specified |channel|.