[Chromoting] Refactor the host policy watcher so that policies can easily be added.

remoting/host/plugin/host_script_object.h

 // Copyright (c) 2012 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_PLUGIN_HOST_SCRIPT_OBJECT_H_
 #define REMOTING_HOST_PLUGIN_HOST_SCRIPT_OBJECT_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 23 matching lines @@
 
 class ChromotingHost;
 class DesktopEnvironment;
 class It2MeHostUserInterface;
 class MutableHostConfig;
 class RegisterSupportHostRequest;
 class SignalStrategy;
 class SupportAccessVerifier;
 
 namespace policy_hack {
-class NatPolicy;
+class PolicyWatcher;
 }  // namespace policy_hack
 
 // NPAPI plugin implementation for remoting host script object.
 // HostNPScriptObject creates threads that are required to run
 // ChromotingHost and starts/stops the host on those threads. When
 // destroyed it sychronously shuts down the host and all threads.
 class HostNPScriptObject : public HostStatusObserver {
  public:
   HostNPScriptObject(NPP plugin, NPObject* parent,
                      PluginMessageLoopProxy::Delegate* plugin_thread_delegate);
@@ skipping 146 matching lines @@
                                const std::string& auth_token,
                                const std::string& auth_service);
   void FinishConnectNetworkThread(const std::string& uid,
                                   const std::string& auth_token,
                                   const std::string& auth_service);
   void DisconnectInternal();
 
   // Callback for ChromotingHost::Shutdown().
   void OnShutdownFinished();
 
+  // Called when a policy is updated.
+  void OnPolicyUpdate(scoped_ptr<base::DictionaryValue> policies);
+
   // Called when the nat traversal policy is updated.
   void OnNatPolicyUpdate(bool nat_traversal_enabled);
 
   void LocalizeStrings(NPObject* localize_func);
 
   // Helper function for executing InvokeDefault on an NPObject that performs
   // a string->string mapping with one optional substitution parameter. Stores
   // the translation in |result| and returns true on success, or leaves it
   // unchanged and returns false on failure.
   bool LocalizeString(NPObject* localize_func, const char* tag,
@@ skipping 82 matching lines @@
   scoped_refptr<ChromotingHost> host_;
   int failed_login_attempts_;
 
   UiStrings ui_strings_;
   base::Lock ui_strings_lock_;
 
   base::WaitableEvent disconnected_event_;
 
   base::Lock nat_policy_lock_;
 
-  scoped_ptr<policy_hack::NatPolicy> nat_policy_;
+  scoped_ptr<policy_hack::PolicyWatcher> policy_watcher_;
 
   // Host the current nat traversal policy setting.
   bool nat_traversal_enabled_;
 
   // Indicates whether or not a policy has ever been read. This is to ensure
   // that on startup, we do not accidentally start a connection before we have
   // queried our policy restrictions.
   bool policy_received_;
 
   // On startup, it is possible to have Connect() called before the policy read
@@ skipping 10 matching lines @@
   // SequencedWorkerPool. Problem is that SequencedWorkerPool relies
   // on MessageLoopProxy::current().
   base::Thread worker_thread_;
 
   DISALLOW_COPY_AND_ASSIGN(HostNPScriptObject);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_PLUGIN_HOST_SCRIPT_OBJECT_H_