Added RefreshPolicies to PolicyService.

chrome/browser/policy/policy_service_impl.cc

 // 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.
 
 #include "chrome/browser/policy/policy_service_impl.h"
 
+#include "base/bind.h"
+#include "base/bind_helpers.h"
 #include "base/observer_list.h"
 #include "base/stl_util.h"
+#include "content/public/browser/browser_thread.h"
+
+using content::BrowserThread;
 
 namespace policy {
 
 struct PolicyServiceImpl::Entry {
   PolicyMap policies;
   ObserverList<PolicyService::Observer, true> observers;
 };
 
 struct PolicyServiceImpl::ProviderData {
   ConfigurationPolicyProvider* provider;
   ConfigurationPolicyObserverRegistrar registrar;
   PolicyMap policies;
+  bool refresh_pending;
 };
 
 PolicyServiceImpl::PolicyServiceImpl(const Providers& providers) {
   initialization_complete_ = true;
   for (size_t i = 0; i < providers.size(); ++i) {
     ConfigurationPolicyProvider* provider = providers[i];
     ProviderData* data = new ProviderData;
     data->provider = provider;
     data->registrar.Init(provider, this);
+    data->refresh_pending = false;
     if (provider->IsInitializationComplete())
       provider->Provide(&data->policies);
     else
       initialization_complete_ = false;
     providers_.push_back(data);
   }
   // There are no observers yet, but calls to GetPolicies() should already get
   // the processed policy values.
-  MergeAndTriggerUpdates();
+  MergeAndTriggerUpdates(false);
 }
 
 PolicyServiceImpl::~PolicyServiceImpl() {
   STLDeleteElements(&providers_);
   STLDeleteValues(&entries_);
 }
 
 void PolicyServiceImpl::AddObserver(PolicyDomain domain,
                                     const std::string& component_id,
                                     PolicyService::Observer* observer) {
@@ skipping 19 matching lines @@
     const std::string& component_id) const {
   PolicyNamespace ns = std::make_pair(domain, component_id);
   EntryMap::const_iterator it = entries_.find(ns);
   return it == entries_.end() ? NULL : &it->second->policies;
 }
 
 bool PolicyServiceImpl::IsInitializationComplete() const {
   return initialization_complete_;
 }
 
+void PolicyServiceImpl::RefreshPolicies(const base::Closure& callback) {
+  if (!callback.is_null())
+    refresh_callbacks_.push_back(callback);
+
+  // Some providers might invoke OnUpdatePolicy synchronously while handling
+  // RefreshPolicies. Flag all with refresh_pending before starting to refresh.
+  ProviderList::iterator it;
+  for (it = providers_.begin(); it != providers_.end(); ++it)
+    (*it)->refresh_pending = true;
+  for (it = providers_.begin(); it != providers_.end(); ++it)
+    (*it)->provider->RefreshPolicies();

[Mattias Nissler (ping if slow), 2012/04/27 09:03:27]

What if providers_ is empty?

[Joao da Silva, 2012/04/27 11:24:12]

Nice catch :-) Fixed.

+}
+
 void PolicyServiceImpl::OnUpdatePolicy(ConfigurationPolicyProvider* provider) {
   ProviderList::iterator it = GetProviderData(provider);
   if (it == providers_.end())
     return;
   provider->Provide(&(*it)->policies);
-  MergeAndTriggerUpdates();
+  bool did_refresh = (*it)->refresh_pending;
+  (*it)->refresh_pending = false;
+  MergeAndTriggerUpdates(did_refresh);
 }
 
 void PolicyServiceImpl::OnProviderGoingAway(
     ConfigurationPolicyProvider* provider) {
   ProviderList::iterator it = GetProviderData(provider);
   if (it == providers_.end())
     return;
+  bool did_refresh = (*it)->refresh_pending;
   delete *it;
   providers_.erase(it);
-  MergeAndTriggerUpdates();
+  MergeAndTriggerUpdates(did_refresh);
 }
 
 PolicyServiceImpl::Entry* PolicyServiceImpl::GetOrCreate(
     const PolicyNamespace& ns) {
   Entry*& entry = entries_[ns];
   if (!entry)
     entry = new Entry;
   return entry;
 }
 
@@ skipping 11 matching lines @@
 void PolicyServiceImpl::MaybeCleanup(const PolicyNamespace& ns) {
   EntryMap::iterator it = entries_.find(ns);
   if (it != entries_.end() &&
       it->second->policies.empty() &&
       it->second->observers.size() == 0) {
     delete it->second;
     entries_.erase(it);
   }
 }
 
-void PolicyServiceImpl::MergeAndTriggerUpdates() {
+void PolicyServiceImpl::MergeAndTriggerUpdates(bool is_refresh) {
   // TODO(joaodasilva): do this for each namespace once the providers also
   // provide policy for more namespaces.
+
   PolicyMap policies;
+  bool refresh_pending = false;
   for (ProviderList::iterator it = providers_.begin();
        it != providers_.end(); ++it) {
     policies.MergeFrom((*it)->policies);
+    refresh_pending |= (*it)->refresh_pending;
   }
 
   Entry* entry = GetOrCreate(std::make_pair(POLICY_DOMAIN_CHROME, ""));
   if (!policies.Equals(entry->policies)) {
     // Swap first, so that observers that call GetPolicies() see the current
     // values.
     entry->policies.Swap(&policies);
     FOR_EACH_OBSERVER(
         PolicyService::Observer,
         entry->observers,
@@ skipping 11 matching lines @@
       }
     }
     if (initialization_complete_) {
       for (EntryMap::iterator i = entries_.begin(); i != entries_.end(); ++i) {
         FOR_EACH_OBSERVER(PolicyService::Observer,
                           i->second->observers,
                           OnPolicyServiceInitialized());
       }
     }
   }
+
+  // Invoke all the callbacks if a refresh has just fully completed.
+  if (is_refresh && !refresh_pending) {
+    std::vector<base::Closure> callbacks;
+    callbacks.swap(refresh_callbacks_);
+    for (size_t i = 0; i < callbacks.size(); ++i)
+      callbacks[i].Run();
+  }
 }
 
 }  // namespace policy