Update VideoFramePump to pass un-changed frames to encoders.

remoting/codec/video_encoder_vpx.h

 // Copyright 2013 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_VPX_H_
 #define REMOTING_CODEC_VIDEO_ENCODER_VPX_H_
 
 #include "base/callback.h"
 #include "base/time/time.h"
 #include "remoting/codec/scoped_vpx_codec.h"
@@ skipping 58 matching lines @@
 
   // VPX image and buffer to hold the actual YUV planes.
   scoped_ptr<vpx_image_t> image_;
   scoped_ptr<uint8[]> image_buffer_;
 
   // Active map used to optimize out processing of un-changed macroblocks.
   scoped_ptr<uint8[]> active_map_;
   int active_map_width_;
   int active_map_height_;
 
+  // Number of "top-off" frames we've encoded since we were last passed a
+  // frame containing an actual change. Used only when encoding with top-off.
+  int topoff_frame_count_;

[Sergey Ulanov, 2015/05/26 21:57:22]

nit: initialize the value here instead of constructor (and maybe move the
initializers for other fields here, except use_vp9_)

[Wez, 2015/06/09 01:46:20]

Done.

Although the compiler was quite happy with me double-initializing. :-/

+
   // Used to help initialize VideoPackets from DesktopFrames.
   VideoEncoderHelper helper_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoEncoderVpx);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CODEC_VIDEO_ENCODER_VP8_H_