Refactor VideoStub interface to accept ownership of video packets.

remoting/protocol/rtp_video_reader.cc

 // Copyright (c) 2012 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 "remoting/protocol/rtp_video_reader.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "remoting/base/constants.h"
 #include "remoting/proto/video.pb.h"
@@ skipping 166 matching lines @@
     return;
   }
 
   // We've found first and last fragments, and we have all fragments in the
   // middle, so we can rebuild fill packet.
   RebuildVideoPacket(first, last);
 }
 
 void RtpVideoReader::RebuildVideoPacket(const PacketsQueue::iterator& first,
                                         const PacketsQueue::iterator& last) {
-  VideoPacket* packet = new VideoPacket();
+  scoped_ptr<VideoPacket> packet(new VideoPacket());
 
   // Set flags.
   if (first->packet->vp8_descriptor().frame_beginning)
     packet->set_flags(packet->flags() | VideoPacket::FIRST_PACKET);
 
   if (last->packet->header().marker)
     packet->set_flags(packet->flags() | VideoPacket::LAST_PACKET);
 
   packet->set_timestamp(first->packet->header().timestamp);
 
@@ skipping 11 matching lines @@
     // packets will be ignored.
   }
 
   packet->mutable_data()->resize(content.total_bytes());
   content.CopyTo(const_cast<char*>(packet->mutable_data()->data()),
                  packet->data().size());
 
   // Set format.
   packet->mutable_format()->set_encoding(VideoPacketFormat::ENCODING_VP8);
 
-  video_stub_->ProcessVideoPacket(
-      packet, base::Bind(&base::DeletePointer<VideoPacket>, packet));
+  video_stub_->ProcessVideoPacket(packet.Pass(), base::Closure());
 
   SendReceiverReportIf();
 }
 
 void RtpVideoReader::SendReceiverReportIf() {
   base::Time now = base::Time::Now();
 
   // Send receiver report only if we haven't sent any bofore, or
   // enough time has passed since the last report.
   if (last_receiver_report_.is_null() ||
       (now - last_receiver_report_).InMilliseconds() >
       kReceiverReportsIntervalMs) {
     RtcpReceiverReport report;
     rtp_reader_.GetReceiverReport(&report);
     rtcp_writer_.SendReport(report);
 
     last_receiver_report_ = now;
   }
 }
 
 }  // namespace protocol
 }  // namespace remoting