Chromium Code Reviews| Index: remoting/codec/video_encoder.h |
| diff --git a/remoting/codec/video_encoder.h b/remoting/codec/video_encoder.h |
| index 81179c4da0862f9acb94f0abad069300d56df3d6..e114dc045e432966c8e794d59a2b58147dbe262c 100644 |
| --- a/remoting/codec/video_encoder.h |
| +++ b/remoting/codec/video_encoder.h |
| @@ -7,19 +7,19 @@ |
| #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. |
| +// A class to perform the task of encoding a continuous stream of images. The |
| +// interface is asynchronous to enable maximum throughput,. |
|
alexeypa (please no reviews)
2013/05/08 22:24:59
nit: remove the comma at the end.
Sergey Ulanov
2013/05/09 18:49:02
Done.
|
| class VideoEncoder { |
| public: |
| @@ -29,15 +29,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. |
| + virtual void Encode(const webrtc::DesktopFrame* frame, |
| const DataAvailableCallback& data_available_callback) = 0; |
| }; |