Replace ScopedThreadProxy with MessageLoopProxy & WeakPtrs.

remoting/host/it2me_host_user_interface.h

Index: remoting/host/it2me_host_user_interface.h
diff --git a/remoting/host/it2me_host_user_interface.h b/remoting/host/it2me_host_user_interface.h
index b054c653983fd637de4b295ff5f8c071db4107a0..c988816224f36cb9b09f1b9ba55d37cae790fb49 100644
--- a/remoting/host/it2me_host_user_interface.h
+++ b/remoting/host/it2me_host_user_interface.h
@@ -11,6 +11,7 @@
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 
 #include "remoting/host/host_user_interface.h"
 
@@ -67,8 +68,9 @@ class It2MeHostUserInterface : public HostUserInterface {
   // the connection.
   scoped_ptr<ContinueWindow> continue_window_;
 
-  // Timer controlling the "continue session" dialog.
-  scoped_ptr<TimerTask> timer_task_;
+  // Weak pointer factory used to abandon the "continue session" timer when
+  // hiding the "continue session" dialog, or tearing down the IT2Me UI.
+  base::WeakPtrFactory<It2MeHostUserInterface> weak_factory_;

[Sergey Ulanov, 2012/05/29 18:58:30]

maybe call it timer_weak_factory_ to indicate that it should not be used as a
regular weak factory.

[Wez, 2012/05/30 00:18:39]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(It2MeHostUserInterface);
 };