Fixed continue window bugs and added 60s auto-shutdown timeout.

remoting/host/desktop_environment.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_HOST_DESKTOP_ENVIRONMENT_H_
 #define REMOTING_HOST_DESKTOP_ENVIRONMENT_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 87 matching lines @@
   // the connection.
   scoped_ptr<ContinueWindow> continue_window_;
 
   // Monitor local inputs to allow remote inputs to be blocked while the local
   // user is trying to do something.
   scoped_ptr<LocalInputMonitor> local_input_monitor_;
 
   bool is_monitoring_local_inputs_;
 
   // Timer controlling the "continue session" dialog.
-  bool continue_timer_started_;
+  enum {INACTIVE, SHOW_DIALOG, HIDE_DIALOG} continue_timer_state_;

[Sergey Ulanov, 2011/08/24 01:56:45]

This is not very readable. Please define type ContinueTimerState and use it
here. Also, add description for each state.

[Jamie, 2011/08/24 17:42:03]

Done. I've also renamed HIDE_DIALOG to SHUTDOWN_HOST, since that's the primary
action taken on timer expiry.

   base::Time continue_timer_target_time_;
 
   scoped_refptr<UIThreadProxy> proxy_;
 
   DISALLOW_COPY_AND_ASSIGN(DesktopEnvironment);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_DESKTOP_ENVIRONMENT_H_