Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(634)

Unified Diff: remoting/codec/video_encoder_verbatim.h

Issue 11195029: Remove ZLib codec support from chromoting host and client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/codec/video_encoder_verbatim.h
diff --git a/remoting/codec/video_encoder_verbatim.h b/remoting/codec/video_encoder_verbatim.h
new file mode 100644
index 0000000000000000000000000000000000000000..ab7602b30a41c13f9c7b29d4957ac859fcef8acb
--- /dev/null
+++ b/remoting/codec/video_encoder_verbatim.h
@@ -0,0 +1,54 @@
+// Copyright (c) 2011 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_VERBATIM_H_
+#define REMOTING_CODEC_VIDEO_ENCODER_VERBATIM_H_
+
+#include "remoting/codec/video_encoder.h"
+#include "remoting/proto/video.pb.h"
+#include "third_party/skia/include/core/SkRect.h"
+
+namespace remoting {
+
+// VideoEncoderVerbatim implements a VideoEncoder that sends image data as is
+// without compression.
Wez 2012/10/17 22:25:16 nit: Clarify the format we mean by "verbatim", e.g
Sergey Ulanov 2012/10/18 01:09:52 Done.
+class VideoEncoderVerbatim : public VideoEncoder {
+ public:
+ VideoEncoderVerbatim();
+ virtual ~VideoEncoderVerbatim();
+
+ void SetMaxPacketSize(int size);
Wez 2012/10/17 22:25:16 nit: Clarify what this is for.
Sergey Ulanov 2012/10/18 01:09:52 Done.
+
+ // VideoEncoder interface.
+ virtual void Encode(
+ scoped_refptr<CaptureData> capture_data,
+ bool key_frame,
+ const DataAvailableCallback& data_available_callback) OVERRIDE;
+
+ private:
+ // Encode a single dirty rect.
+ void EncodeRect(const SkIRect& rect, bool last);
+
+ // Marks a packet as the first in a series of rectangle updates.
Wez 2012/10/17 22:25:16 nit: How is |rect| used?
Sergey Ulanov 2012/10/18 01:09:52 Done.
+ void PrepareUpdateStart(const SkIRect& rect, VideoPacket* packet);
+
+ // Retrieves a pointer to the output buffer in |update| used for storing the
Wez 2012/10/17 22:25:16 What is |update|?
Sergey Ulanov 2012/10/18 01:09:52 That was old stale comment from row_based encoder.
+ // encoded rectangle data. Will resize the buffer to |size|.
+ uint8* GetOutputBuffer(VideoPacket* packet, size_t size);
+
+ // Submit |message| to |callback_|.
Wez 2012/10/17 22:25:16 nit: |message| -> |packet|
Sergey Ulanov 2012/10/18 01:09:52 Done.
+ void SubmitMessage(VideoPacket* packet, size_t rect_index);
+
+ scoped_refptr<CaptureData> capture_data_;
+ DataAvailableCallback callback_;
+
+ // The most recent screen size.
+ SkISize screen_size_;
+
+ int packet_size_;
Wez 2012/10/17 22:25:16 nit: max_packet_size_
Sergey Ulanov 2012/10/18 01:09:52 Done.
+};
+
+} // namespace remoting
+
+#endif // REMOTING_CODEC_VIDEO_ENCODER_VERBATIM_H_

Powered by Google App Engine
This is Rietveld 408576698