Properly handle screen recorder shutdown in ChromotingHost.

remoting/host/screen_recorder_unittest.cc

 // Copyright (c) 2011 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 "base/bind.h"
 #include "base/message_loop.h"
 #include "base/task.h"
 #include "remoting/base/base_mock_objects.h"
 #include "remoting/host/host_mock_objects.h"
 #include "remoting/host/screen_recorder.h"

[dmac, 2011/08/11 23:56:34]

NIT: this include should be on top...

[Sergey Ulanov, 2011/08/12 00:29:48]

Done.

 #include "remoting/proto/video.pb.h"
 #include "remoting/protocol/protocol_mock_objects.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using ::remoting::protocol::MockConnectionToClient;
 using ::remoting::protocol::MockConnectionToClientEventHandler;
 using ::remoting::protocol::MockHostStub;
 using ::remoting::protocol::MockVideoStub;
 
@@ skipping 163 matching lines @@
   // By default delete the arguments when ProcessVideoPacket is received.
   EXPECT_CALL(video_stub, ProcessVideoPacket(_, _))
       .WillRepeatedly(FinishSend());
 
   // For the first time when ProcessVideoPacket is received we stop the
   // ScreenRecorder.
   EXPECT_CALL(video_stub, ProcessVideoPacket(_, _))
       .WillOnce(DoAll(
           FinishSend(),
           StopScreenRecorder(record_,
-                             NewRunnableFunction(&QuitMessageLoop,
-                                                 &message_loop_))))
+                             base::Bind(&QuitMessageLoop, &message_loop_))))
       .RetiresOnSaturation();
 
   // Add the mock client connection to the session.
   record_->AddConnection(connection_);
 
   // Start the recording.
   record_->Start();
   message_loop_.Run();
 }
 
 TEST_F(ScreenRecorderTest, StopWithoutStart) {
-  record_->Stop(NewRunnableFunction(&QuitMessageLoop, &message_loop_));
+  record_->Stop(base::Bind(&QuitMessageLoop, &message_loop_));
   message_loop_.Run();
 }
 
 }  // namespace remoting