Use webrtc::DesktopCapturer for screen capturer implementation.

remoting/codec/video_encoder.h

 // 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.
 
 #ifndef REMOTING_CODEC_VIDEO_ENCODER_H_
 #define REMOTING_CODEC_VIDEO_ENCODER_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
-#include "media/base/data_buffer.h"
 
-namespace media {
-class ScreenCaptureData;
-}  // namespace media
+class SkRegion;
+
+namespace webrtc {
+class DesktopFrame;
+}  // namespace webrtc
 
 namespace remoting {
 
 class VideoPacket;
 
 // A class to perform the task of encoding a continous stream of
 // images.
 // This class operates asynchronously to enable maximum throughput.

[Wez, 2013/04/26 18:48:14]

nit: This is an interface, not a class, so unless we intend the interface to be
inherently asynchronous, this doesn't apply.

[Sergey Ulanov, 2013/05/07 22:25:50]

Done.

 class VideoEncoder {
  public:
 
   // DataAvailableCallback is called one or more times, for each chunk the
   // underlying video encoder generates.
   typedef base::Callback<void(scoped_ptr<VideoPacket>)> DataAvailableCallback;
 
   virtual ~VideoEncoder() {}
 
-  // 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<media::ScreenCaptureData> capture_data,
-                      bool key_frame,
+  // Encode an image stored in |frame|. Doesn't take ownership of |frame|. When
+  // encoded data is available, partial or full |data_available_callback| is
+  // called.

[Wez, 2013/04/26 18:48:14]

IIRC the implementations are synchronous, so you could make that part of the
interface contract.

If we want it to stay possibly-asynchronous then you need the |frame| to be
owned by the encoder until it has finished processing it, at which point it
could return it via the callback (possibly with no actual VideoPacket to write
out).

[Sergey Ulanov, 2013/05/07 22:25:50]

Not related to this CL.

+  virtual void Encode(webrtc::DesktopFrame* frame,

[Wez, 2013/04/26 18:48:14]

If this isn't being passed, it can be const DesktopFrame*, I think?

[Sergey Ulanov, 2013/05/07 22:25:50]

Done.

                       const DataAvailableCallback& data_available_callback) = 0;
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CODEC_VIDEO_ENCODER_H_