Remove done notifications from incoming message handlers.

remoting/protocol/audio_reader.cc

Index: remoting/protocol/audio_reader.cc
diff --git a/remoting/protocol/audio_reader.cc b/remoting/protocol/audio_reader.cc
index 02f8eb1c3e7e1bc64f43f37c322ddb990ad90c60..d197a83d57d81aa45a6473d45ce80de310088d20 100644
--- a/remoting/protocol/audio_reader.cc
+++ b/remoting/protocol/audio_reader.cc
@@ -15,12 +15,15 @@ namespace protocol {
 
 AudioReader::AudioReader(AudioStub* audio_stub)
     : ChannelDispatcherBase(kAudioChannelName),
-      parser_(base::Bind(&AudioStub::ProcessAudioPacket,
-                         base::Unretained(audio_stub)),
-              reader()) {
-}
+      audio_stub_(audio_stub),
+      parser_(base::Bind(&AudioReader::OnAudioPacket, base::Unretained(this)),
+              reader()) {}
+
+AudioReader::~AudioReader() {}
 
-AudioReader::~AudioReader() {
+void AudioReader::OnAudioPacket(scoped_ptr<AudioPacket> audio_packet) {
+  audio_stub_->ProcessAudioPacket(std::move(audio_packet),
+                                  base::Bind(&base::DoNothing));

[Jamie, 2016/01/29 23:48:12]

Is it possible to remove the callback from ProcessAudioPacket as well (rather
than introducing this helper method) or is it required elsewhere?

[Sergey Ulanov, 2016/01/30 00:14:47]

It's still used on the sender side. Without it AudioPump will keep pumping audio
when the sender is blocked. 

 }
 
 }  // namespace protocol