Properly handle screen recorder shutdown in ChromotingHost.

remoting/host/chromoting_host.h

 // 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.
 
 #ifndef REMOTING_CHROMOTING_HOST_H_
 #define REMOTING_CHROMOTING_HOST_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 164 matching lines @@
 
   std::string GenerateHostAuthToken(const std::string& encoded_client_token);
 
   int AuthenticatedClientsCount() const;
 
   void EnableCurtainMode(bool enable);
 
   void ProcessPreAuthentication(
       const scoped_refptr<protocol::ConnectionToClient>& connection);
 
+  void StopScreenRecorder();
+  void OnScreenRecorderStopped();
+
   // The following methods are called during shutdown.
   void ShutdownNetwork();
   void ShutdownRecorder();
   void ShutdownFinish();
 
   // Parameters specified when the host was created.
   ChromotingHostContext* context_;
   DesktopEnvironment* desktop_environment_;
   scoped_refptr<MutableHostConfig> config_;
   scoped_ptr<AccessVerifier> access_verifier_;
@@ skipping 10 matching lines @@
   ClientList clients_;
 
   // Session manager for the host process.
   scoped_refptr<ScreenRecorder> recorder_;
 
   // Tracks the internal state of the host.
   // This variable is written on the main thread of ChromotingHostContext
   // and read by jingle thread.
   State state_;
 
+  // Number of screen recorders that are currently being stopped.

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

Maybe expand this comment to explain the case that you explained to me? eg we
are we worried about this?

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

Done.

+  int stopping_recorders_;
+
   // Lock is to lock the access to |state_|.
   base::Lock lock_;
 
   // Configuration of the protocol.
   scoped_ptr<protocol::CandidateSessionConfig> protocol_config_;
 
   bool is_curtained_;
 
   // Whether or not the host is running in "IT2Me" mode, in which connections
   // are pre-authenticated, and hence the local login challenge can be bypassed.
   bool is_it2me_;
 
   std::string access_code_;
 
   // Stores list of tasks that should be executed when we finish
   // shutdown. Used only while |state_| is set to kStopping.
   std::vector<Task*> shutdown_tasks_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingHost);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CHROMOTING_HOST_H_