Enforce the RemoteAccessHostRequireCurtain policy on all platforms.

remoting/host/curtaining_host_observer.h

Index: remoting/host/curtaining_host_observer.h
diff --git a/remoting/host/curtaining_host_observer.h b/remoting/host/curtaining_host_observer.h
index 92c740add7f070ff85f82443a44f9cffc136f82e..7ab2d58119cb718767b84f1478dd3a88c84e5fd7 100644
--- a/remoting/host/curtaining_host_observer.h
+++ b/remoting/host/curtaining_host_observer.h
@@ -5,6 +5,7 @@
 #ifndef REMOTING_HOST_CURTAINING_HOST_OBSERVER_H_
 #define REMOTING_HOST_CURTAINING_HOST_OBSERVER_H_
 
+#include <set>
 #include <string>
 
 #include "base/basictypes.h"
@@ -24,13 +25,19 @@ class CurtainingHostObserver : public HostStatusObserver {
                          scoped_refptr<ChromotingHost> host);
   virtual ~CurtainingHostObserver();
 
-  // From HostStatusObserver.
+  // Enables/disabled curtaining when one or more clients are connected.

[Jamie, 2012/11/07 00:27:49]

Nit: s/disabled/disables/

[Wez, 2012/11/07 02:37:32]

Done.

+  // Takes immediate effect if clients are already connected.
+  void SetEnableCurtaining(bool enable);

[Jamie, 2012/11/07 00:27:49]

Check with Renato, or see his previous CL around curtain mode. I think I
suggested moving the enable state into the observer, but he pointed out it
causes a problem in a corner case.

[rmsousa, 2012/11/07 00:33:25]

Having it here is fine, as long as there's another copy somewhere permanent (in
this change, remoting_me2me_host) to be reapplied when the host observer is
deleted/recreated.

+
+  // HostStatusObserver interface.
   virtual void OnClientAuthenticated(const std::string& jid) OVERRIDE;
   virtual void OnClientDisconnected(const std::string& jid) OVERRIDE;
 
  private:
   CurtainMode* curtain_;
   scoped_refptr<ChromotingHost> host_;
+  std::set<std::string> active_clients_;
+  bool enable_curtaining_;
 };
 
 }  // namespace remoting