Fix VideoFramePump to send keep-alive packets only after the first real frame.

remoting/host/video_frame_pump.cc

Index: remoting/host/video_frame_pump.cc
diff --git a/remoting/host/video_frame_pump.cc b/remoting/host/video_frame_pump.cc
index 1f11264b786e180eb4e2154a96629600d72a3068..6451313af3e560ac0271091dba7b0c00d3307fcc 100644
--- a/remoting/host/video_frame_pump.cc
+++ b/remoting/host/video_frame_pump.cc
@@ -60,7 +60,12 @@ VideoFramePump::VideoFramePump(
       capturer_(capturer.Pass()),
       encoder_(encoder.Pass()),
       video_stub_(video_stub),
-      keep_alive_timer_(true, true),
+      keep_alive_timer_(
+          FROM_HERE,
+          base::TimeDelta::FromMilliseconds(kKeepAlivePacketIntervalMs),
+          base::Bind(&VideoFramePump::SendKeepAlivePacket,
+                     base::Unretained(this)),
+          false),

[Wez, 2015/02/18 22:59:35]

You've changed this from repeating to non-repeating - was that intentional?

[Sergey Ulanov, 2015/02/18 23:31:41]

Yes - it's mentioned in the CL description. It's not necessary to fix the bug,
but I think it needs to be fixed.

       capture_scheduler_(base::Bind(&VideoFramePump::CaptureNextFrame,
                                     base::Unretained(this))),
       latest_event_timestamp_(0),
@@ -70,10 +75,6 @@ VideoFramePump::VideoFramePump(
 
   capturer_->Start(this);
   capture_scheduler_.Start();
-
-  keep_alive_timer_.Start(
-      FROM_HERE, base::TimeDelta::FromMilliseconds(kKeepAlivePacketIntervalMs),
-      base::Bind(&VideoFramePump::SendKeepAlivePacket, base::Unretained(this)));
 }
 
 VideoFramePump::~VideoFramePump() {