Refactor VideoStub interface to accept ownership of video packets.

remoting/protocol/rtp_video_reader_unittest.cc

Index: remoting/protocol/rtp_video_reader_unittest.cc
diff --git a/remoting/protocol/rtp_video_reader_unittest.cc b/remoting/protocol/rtp_video_reader_unittest.cc
index fc1f79f697a7ff089de679028dff0035c4729b0d..b5940defd75723e7cb8e85e730a6b33661fe8270 100644
--- a/remoting/protocol/rtp_video_reader_unittest.cc
+++ b/remoting/protocol/rtp_video_reader_unittest.cc
@@ -25,11 +25,12 @@ class RtpVideoReaderTest : public testing::Test,
                            public VideoStub {
  public:
   // VideoStub interface.
-  virtual void ProcessVideoPacket(const VideoPacket* video_packet,
+  virtual void ProcessVideoPacket(scoped_ptr<VideoPacket> video_packet,
                                   const base::Closure& done) {
     received_packets_.push_back(VideoPacket());
     received_packets_.back() = *video_packet;
-    done.Run();
+    if (!done.is_null())
+      done.Run();
   }
 
   virtual int GetPendingPackets() {