Switch remoting/protocol to new callbacks

remoting/protocol/video_stub.h

Index: remoting/protocol/video_stub.h
diff --git a/remoting/protocol/video_stub.h b/remoting/protocol/video_stub.h
index d9c3714c620c5fbd8507c55335b76c896d180590..16e467709dba24f1097bc78441fe87c16f6b7581 100644
--- a/remoting/protocol/video_stub.h
+++ b/remoting/protocol/video_stub.h
@@ -5,7 +5,7 @@
 #ifndef REMOTING_PROTOCOL_VIDEO_STUB_H_
 #define REMOTING_PROTOCOL_VIDEO_STUB_H_
 
-class Task;
+#include "base/callback.h"

[awong, 2011/10/04 01:18:07]

FYI, You can still forward declare base::Closure if you want.  But #including
works just as well.  Your choice.

[Sergey Ulanov, 2011/10/04 01:27:29]

I tried it, but GCC doesn't like it because Closure is a typedef, not a class:
../../remoting/protocol/video_stub.h:9: error: using typedef-name
‘base::Closure’ after ‘class’
../../base/callback.h:551: error: ‘base::Closure’ has a previous declaration
here

 
 namespace remoting {
 
@@ -21,7 +21,7 @@ class VideoStub {
   // video packets in protobuf stream. It should not be used here. Add another
   // struct and use it to represent video packets internally.
   virtual void ProcessVideoPacket(const VideoPacket* video_packet,
-                                  Task* done) = 0;
+                                  const base::Closure& done) = 0;
 
   // Returns number of packets currently pending in the buffer.
   virtual int GetPendingPackets() = 0;