Use webrtc::DesktopCapturer for screen capturer implementation.

remoting/codec/video_encoder.h

Index: remoting/codec/video_encoder.h
diff --git a/remoting/codec/video_encoder.h b/remoting/codec/video_encoder.h
index 81179c4da0862f9acb94f0abad069300d56df3d6..7520b454fe330ae15fbe12760e8f5f00a0f5a113 100644
--- a/remoting/codec/video_encoder.h
+++ b/remoting/codec/video_encoder.h
@@ -7,11 +7,12 @@
 
 #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 {
 
@@ -29,15 +30,10 @@ class VideoEncoder {
 
   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;
 };