Refactor VideoStub interface to accept ownership of video packets.

Previously VideoStub::ProcessVideoPacket() did not accept ownership of
video packets. Instead the done callback was responsible for deleting
the packet.


Review URL: http://codereview.chromium.org/9827006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129355 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 3 deletions

diff --git a/remoting/client/rectangle_update_decoder.cc b/remoting/client/rectangle_update_decoder.cc
index 8eeebd5..0ad0422 100644
--- a/remoting/client/rectangle_update_decoder.cc
+++ b/remoting/client/rectangle_update_decoder.cc
@@ -52,12 +52,12 @@ void RectangleUpdateDecoder::Initialize(const SessionConfig& config) {
   }
 }
 
-void RectangleUpdateDecoder::DecodePacket(const VideoPacket* packet,
+void RectangleUpdateDecoder::DecodePacket(scoped_ptr<VideoPacket> packet,
                                           const base::Closure& done) {
   if (!message_loop_->BelongsToCurrentThread()) {
     message_loop_->PostTask(
         FROM_HERE, base::Bind(&RectangleUpdateDecoder::DecodePacket,
-                              this, packet, done));
+                              this, base::Passed(&packet), done));
     return;
   }
 
@@ -91,7 +91,7 @@ void RectangleUpdateDecoder::DecodePacket(const VideoPacket* packet,
     return;
   }
 
-  if (decoder_->DecodePacket(packet) == Decoder::DECODE_DONE)
+  if (decoder_->DecodePacket(packet.get()) == Decoder::DECODE_DONE)
     SchedulePaint();
 }