Future proof against things like http://crbug.com/91521

remoting/client/client_context.h

Index: remoting/client/client_context.h
diff --git a/remoting/client/client_context.h b/remoting/client/client_context.h
index 38c60880c49f5d6d64a5684813dd01fa7203d3b3..01f320db4ab6161c200a185e3680c2d2f0700086 100644
--- a/remoting/client/client_context.h
+++ b/remoting/client/client_context.h
@@ -13,11 +13,11 @@
 namespace remoting {
 
 // A class that manages threads and running context for the chromoting client
-// process.
+// process. This class is not designed to be subclassed.
 class ClientContext {
  public:
   ClientContext();
-  virtual ~ClientContext();
+  ~ClientContext();
 
   void Start();

[awong, 2011/08/03 22:14:23]

Add comment hat Start and Stop must be called from the same thread?

   void Stop();
@@ -40,6 +40,9 @@ class ClientContext {
   // A thread that handles all decode operations.
   base::Thread decode_thread_;
 
+  // True if Start() was called on the context.
+  bool started_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientContext);
 };