Remove done notifications from incoming message handlers.

remoting/protocol/client_video_dispatcher.cc

Index: remoting/protocol/client_video_dispatcher.cc
diff --git a/remoting/protocol/client_video_dispatcher.cc b/remoting/protocol/client_video_dispatcher.cc
index 46df38f5eb209ae181099ffd81c51f02df39a0f1..e913700f884ea0ab5c284c72dff771488cd7262d 100644
--- a/remoting/protocol/client_video_dispatcher.cc
+++ b/remoting/protocol/client_video_dispatcher.cc
@@ -31,22 +31,17 @@ ClientVideoDispatcher::ClientVideoDispatcher(VideoStub* video_stub)
       parser_(base::Bind(&ClientVideoDispatcher::ProcessVideoPacket,
                          base::Unretained(this)),
               reader()),
-      weak_factory_(this) {
-}
+      weak_factory_(this) {}
 
-ClientVideoDispatcher::~ClientVideoDispatcher() {
-}
+ClientVideoDispatcher::~ClientVideoDispatcher() {}
 
 void ClientVideoDispatcher::ProcessVideoPacket(
-    scoped_ptr<VideoPacket> video_packet,
-    const base::Closure& done) {
-  base::ScopedClosureRunner done_runner(done);
-
+    scoped_ptr<VideoPacket> video_packet) {
   int frame_id = video_packet->frame_id();
 
   if (!video_packet->has_frame_id()) {
     video_stub_->ProcessVideoPacket(std::move(video_packet),
-                                    done_runner.Release());
+                                    base::Bind(&base::DoNothing));

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

Similarly here, can you remove the callback parameter?

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

VideoStub is used on the sender side, and this callback is still useful there,
particularly to measure send latency.

     return;
   }