Added ConfigurationPolicyProvider::RefreshPolicies.

chrome/browser/policy/asynchronous_policy_provider.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 CHROME_BROWSER_POLICY_ASYNCHRONOUS_POLICY_PROVIDER_H_
 #define CHROME_BROWSER_POLICY_ASYNCHRONOUS_POLICY_PROVIDER_H_
 #pragma once
 
 #include "base/memory/ref_counted.h"
+#include "base/memory/weak_ptr.h"
 #include "base/threading/non_thread_safe.h"
 #include "chrome/browser/policy/configuration_policy_provider.h"
 
 namespace policy {
 
 class AsynchronousPolicyLoader;
 
 // Policy provider that loads policy asynchronously. Providers should subclass
 // from this class if loading the policy requires disk access or must for some
 // other reason be performed on the file thread. The actual logic for loading
@@ skipping 10 matching lines @@
 
     virtual DictionaryValue* Load() = 0;
   };
 
   // Assumes ownership of |loader|.
   AsynchronousPolicyProvider(
       const PolicyDefinitionList* policy_list,
       scoped_refptr<AsynchronousPolicyLoader> loader);
   virtual ~AsynchronousPolicyProvider();
 
-  // Force a policy reload from the underlying data source.
-  void ForceReload();
-
- protected:
   // ConfigurationPolicyProvider implementation.
   virtual bool ProvideInternal(PolicyMap* map) OVERRIDE;
+  virtual void RefreshPolicies() OVERRIDE;
+
+ private:
+  // Used to trigger a Reload on |loader| on the FILE thread.
+  static void PostReloadOnFILE(scoped_refptr<AsynchronousPolicyLoader> loader);

[Mattias Nissler (ping if slow), 2011/11/18 14:49:32]

In these cases we usually s/FILE/FileThread/:

$ git grep 'FileThread' | wc -l
335
$ git grep 'FILE(' | wc -l
29

[Mattias Nissler (ping if slow), 2011/11/18 14:49:32]

Pass loader as plain pointer and base::Bind will take care of the refcounting.

[Joao da Silva, 2011/11/18 15:39:18]

Done.

[Joao da Silva, 2011/11/18 15:39:18]

Done.

+
+  // Used to notify UI that a reload task has been submitted.
+  void OnReloadPosted();
+
+  // Callback from the loader. This is invoked whenever the loader has completed
+  // a reload of the policies.
+  void OnLoaderReloaded();
 
   // The loader object used internally.
   scoped_refptr<AsynchronousPolicyLoader> loader_;
 
- private:
+  // Number of refreshes requested whose reload is still pending. Used to only
+  // fire notifications when all pending refreshes are done.
+  int pending_refreshes_;
+
+  // Used to post tasks to self on UI.
+  base::WeakPtrFactory<AsynchronousPolicyProvider> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(AsynchronousPolicyProvider);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_ASYNCHRONOUS_POLICY_PROVIDER_H_