Refactor VideoStub interface to accept ownership of video packets.

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 064ce1fb56d40e16b7077017fe8409308196d8c5..86875a175ab264d087e44247ac298abdc577334c 100644
--- a/remoting/protocol/video_stub.h
+++ b/remoting/protocol/video_stub.h
@@ -6,6 +6,7 @@
 #define REMOTING_PROTOCOL_VIDEO_STUB_H_
 
 #include "base/callback_forward.h"
+#include "base/memory/scoped_ptr.h"
 
 namespace remoting {
 
@@ -20,7 +21,7 @@ class VideoStub {
   // TODO(sergeyu): VideoPacket is the protobuf message that is used to send
   // 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,
+  virtual void ProcessVideoPacket(scoped_ptr<VideoPacket> video_packet,
                                   const base::Closure& done) = 0;
 
   // Returns number of packets currently pending in the buffer.