Interface with webrtc through encoded frames

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 9da447c1336f17394dd54b6d5273f81c4656f48a..276be5efc0cfe65e221be42d28074a83a83dad16 100644
--- a/remoting/codec/video_encoder.h
+++ b/remoting/codec/video_encoder.h
@@ -5,6 +5,8 @@
 #ifndef REMOTING_CODEC_VIDEO_ENCODER_H_
 #define REMOTING_CODEC_VIDEO_ENCODER_H_
 
+#include <stdint.h>
+
 #include "base/memory/scoped_ptr.h"
 
 namespace webrtc {
@@ -19,6 +21,7 @@ class VideoPacket;
 // interface is asynchronous to enable maximum throughput.
 class VideoEncoder {
  public:
+  enum Flags { kRequestKeyFrame = 1 << 0 };

[Sergey Ulanov, 2016/04/07 21:05:39]

Call it REQUEST_KEY_FRAME. See
https://www.chromium.org/developers/coding-style#TOC-Naming

[Irfan, 2016/04/12 21:13:11]

Done.

   virtual ~VideoEncoder() {}
 
   // Request that the encoder provide lossless encoding, or color, if possible.
@@ -29,7 +32,8 @@ class VideoEncoder {
   // then the encoder may return a packet (e.g. to top-off previously-encoded
   // portions of the frame to higher quality) or return nullptr to indicate that
   // there is no work to do.
-  virtual scoped_ptr<VideoPacket> Encode(const webrtc::DesktopFrame& frame) = 0;
+  virtual scoped_ptr<VideoPacket> Encode(const webrtc::DesktopFrame& frame,
+                                         uint32_t flags = 0) = 0;

[Sergey Ulanov, 2016/04/07 21:05:39]

style guide doesn't allow optional arguments.

You can overload this function to avoid changing all callers:
scoped_ptr<VideoPacket> Encode(const webrtc::DesktopFrame& frame) {
  return Encode(frame, 0);
}

But there shouldn't be many call sites, so it should be possible to update all
of thme.

[Irfan, 2016/04/12 21:13:11]

Done.

 };
 
 }  // namespace remoting