Refactor VideoStub interface to accept ownership of video packets.

remoting/protocol/rtp_video_reader_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
 #include "base/string_number_conversions.h"
 #include "net/base/io_buffer.h"
 #include "remoting/proto/video.pb.h"
 #include "remoting/protocol/fake_session.h"
 #include "remoting/protocol/rtp_utils.h"
 #include "remoting/protocol/rtp_video_reader.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using net::IOBuffer;
 using std::vector;
 
 namespace remoting {
 namespace protocol {
 
 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() {
     return 0;
   }
 
  protected:
   struct FragmentInfo {
     int sequence_number;
     int timestamp;
@@ skipping 326 matching lines @@
 
   ExpectedPacket expected[] = {
     { 123, VideoPacket::FIRST_PACKET | VideoPacket::LAST_PACKET, 0, 30 },
     { 223, VideoPacket::FIRST_PACKET | VideoPacket::LAST_PACKET, 40, 50 },
   };
   CheckResults(expected, arraysize(expected));
 }
 
 }  // namespace protocol
 }  // namespace remoting