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

Unified Diff: remoting/codec/video_encoder_verbatim.h

Issue 530243002: Move common VideoPacket initialization into VideoEncoderHelper. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add unit tests Created 6 years, 3 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
index 54a8a033fa978370eacb2919d0bdf27f6806bbe6..0005a4ae1c96299ac34f09e567d56a6a03cb067a 100644
--- a/remoting/codec/video_encoder_verbatim.h
+++ b/remoting/codec/video_encoder_verbatim.h
@@ -6,8 +6,7 @@
#define REMOTING_CODEC_VIDEO_ENCODER_VERBATIM_H_
#include "remoting/codec/video_encoder.h"
-#include "remoting/proto/video.pb.h"
-#include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
+#include "remoting/codec/video_encoder_helper.h"
namespace remoting {
@@ -23,12 +22,9 @@ class VideoEncoderVerbatim : public VideoEncoder {
const webrtc::DesktopFrame& frame) OVERRIDE;
private:
- // Allocates a buffer of the specified |size| inside |packet| and returns the
- // pointer to it.
- uint8* GetOutputBuffer(VideoPacket* packet, size_t size);
+ VideoEncoderHelper helper_;
- // The most recent screen size. Used to detect screen size changes.
- webrtc::DesktopSize screen_size_;
+ DISALLOW_COPY_AND_ASSIGN(VideoEncoderVerbatim);
};
} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698