Use scoped_ptr<> to pass ownership in more places.

remoting/base/encoder.h

 // Copyright (c) 2010 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.
 
 #ifndef REMOTING_BASE_ENCODER_H_
 #define REMOTING_BASE_ENCODER_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "media/base/data_buffer.h"
 
 namespace media {
   class DataBuffer;
 }
 
 namespace remoting {
 
 class CaptureData;
 class VideoPacket;
 
 // A class to perform the task of encoding a continous stream of
 // images.
 // This class operates asynchronously to enable maximum throughput.
 class Encoder {
  public:
 
   // DataAvailableCallback is called as blocks of data are made available
   // from the encoder. Data made available by the encoder is in the form
   // of HostMessage to reduce the amount of memory copies.
-  // The callback takes ownership of the HostMessage and is responsible for
-  // deleting it.
-  typedef base::Callback<void(VideoPacket*)> DataAvailableCallback;
+  typedef base::Callback<void(scoped_ptr<VideoPacket>)> DataAvailableCallback;
 
   virtual ~Encoder() {}
 
   // Encode an image stored in |capture_data|.
   //
   // If |key_frame| is true, the encoder should not reference
   // previous encode and encode the full frame.
   //
   // When encoded data is available, partial or full |data_available_callback|
   // is called.
   virtual void Encode(scoped_refptr<CaptureData> capture_data,
                       bool key_frame,
                       const DataAvailableCallback& data_available_callback) = 0;
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_BASE_ENCODER_H_