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

remoting/host/client_session_unittest.cc

 // Copyright (c) 2012 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.
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
@@ skipping 366 matching lines @@
 
     EXPECT_CALL(*input_injector_, InjectClipboardEvent(EqualsClipboardEvent(
                                       kMimeTypeTextUtf8, "c")));
     EXPECT_CALL(*input_injector_, InjectKeyEvent(EqualsUsbEvent(3, true)));
     EXPECT_CALL(*input_injector_, InjectMouseEvent(EqualsMouseEvent(300, 301)));
     EXPECT_CALL(*input_injector_, InjectKeyEvent(EqualsUsbEvent(3, false)));
   }
 
   ConnectClientSession();
 
-  // With for the first frame.
+  // Wait for the first frame.
   run_loop.Run();
 
   // Inject test events that are expected to be injected.
   protocol::ClipboardEvent clipboard_event;
   clipboard_event.set_mime_type(kMimeTypeTextUtf8);
   clipboard_event.set_data("a");
   connection_->clipboard_stub()->InjectClipboardEvent(clipboard_event);
 
   protocol::KeyEvent key_event;
   key_event.set_pressed(true);
@@ skipping 40 matching lines @@
 
   mouse_event.set_x(300);
   mouse_event.set_y(301);
   connection_->input_stub()->InjectMouseEvent(mouse_event);
 
   client_session_->DisconnectSession();
   client_session_->OnConnectionClosed(connection_, protocol::OK);
   client_session_.reset();
 }
 
-// crbug.com/458691
-TEST_F(ClientSessionTest, DISABLED_LocalInputTest) {
+TEST_F(ClientSessionTest, LocalInputTest) {
   CreateClientSession();
 
   protocol::MouseEvent mouse_event1;
   mouse_event1.set_x(100);
   mouse_event1.set_y(101);
   protocol::MouseEvent mouse_event2;
   mouse_event2.set_x(200);
   mouse_event2.set_y(201);
   protocol::MouseEvent mouse_event3;
   mouse_event3.set_x(300);
@@ skipping 35 matching lines @@
   EXPECT_CALL(*input_injector_, InjectMouseEvent(EqualsMouseEvent(100, 101)))
       .InSequence(s);
   EXPECT_CALL(*input_injector_, InjectMouseEvent(EqualsMouseEvent(200, 201)))
       .InSequence(s);
   EXPECT_CALL(session_event_handler_, OnSessionClosed(_))
       .InSequence(s);
 
   ConnectClientSession();
 }
 
-// crbug.com/458691
-TEST_F(ClientSessionTest, DISABLED_RestoreEventState) {
+TEST_F(ClientSessionTest, RestoreEventState) {
   CreateClientSession();
 
   protocol::KeyEvent key1;
   key1.set_pressed(true);
   key1.set_usb_keycode(1);
 
   protocol::KeyEvent key2;
   key2.set_pressed(true);
   key2.set_usb_keycode(2);
 
@@ skipping 35 matching lines @@
       .InSequence(s);
   EXPECT_CALL(*input_injector_, InjectMouseEvent(EqualsMouseButtonEvent(
       protocol::MouseEvent::BUTTON_LEFT, false)))
       .InSequence(s);
   EXPECT_CALL(session_event_handler_, OnSessionClosed(_))
       .InSequence(s);
 
   ConnectClientSession();
 }
 
-// crbug.com/458691
-TEST_F(ClientSessionTest, DISABLED_ClampMouseEvents) {
+TEST_F(ClientSessionTest, ClampMouseEvents) {
   CreateClientSession();
 
   Expectation authenticated =
       EXPECT_CALL(session_event_handler_, OnSessionAuthenticated(_))
           .WillOnce(Return(true));
   EXPECT_CALL(*input_injector_, StartPtr(_))
       .After(authenticated);
   EXPECT_CALL(session_event_handler_, OnSessionChannelsConnected(_))
       .After(authenticated);
   EXPECT_CALL(session_event_handler_, OnSessionClosed(_))
@@ skipping 214 matching lines @@
 
   DisconnectClientSession();
   StopClientSession();
 
   // ext1 was instantiated and wrapped the video capturer.
   EXPECT_TRUE(extension.was_instantiated());
   EXPECT_TRUE(extension.has_wrapped_video_capturer());
 }
 
 }  // namespace remoting