Added a PolicyDomainDescriptor.

chrome/browser/policy/async_policy_loader.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/async_policy_loader.h"
 
 #include "base/bind.h"
 #include "chrome/browser/policy/policy_bundle.h"
 #include "content/public/browser/browser_thread.h"
 
@@ skipping 37 matching lines @@
   }
 
   scoped_ptr<PolicyBundle> bundle(Load());
 
   // Check if there was a modification while reading.
   if (!force && !IsSafeToReload(now, &delay)) {
     ScheduleNextReload(delay);
     return;
   }
 
+  // Filter out mismatching policies.
+  for (DescriptorMap::iterator it = descriptor_map_.begin();
+       it != descriptor_map_.end(); ++it) {
+    it->second->FilterBundle(bundle.get());

[Mattias Nissler (ping if slow), 2013/05/15 10:24:06]

I don't think this is a good idea - the idea was that chrome://policy does show
misconfigured policy. I guess a better point for filtering would be at the
PolicyService or ConfigurationPolicyPrefStore level.

[Joao da Silva, 2013/05/19 13:17:55]

Note that this only applies to non-CHROME policy, so typos will still show up.

Typos for non-CHROME policy are trickier to find, since the loaders use the
schema information to determine which names to lookup. That will be figured out
in future work, since about:policy doesn't show extension policy for now.

+  }
+
   update_callback_.Run(bundle.Pass());
   ScheduleNextReload(TimeDelta::FromSeconds(kReloadIntervalSeconds));
 }
 
+void AsyncPolicyLoader::RegisterPolicyDomain(
+    scoped_refptr<const PolicyDomainDescriptor> descriptor) {
+  if (descriptor->domain() != POLICY_DOMAIN_CHROME) {
+    descriptor_map_[descriptor->domain()] = descriptor;
+    Reload(true);
+  }
+}
+
 scoped_ptr<PolicyBundle> AsyncPolicyLoader::InitialLoad() {
   // This is the first load, early during startup. Use this to record the
   // initial |last_modification_time_|, so that potential changes made before
   // installing the watches can be detected.
   last_modification_time_ = LastModificationTime();
   return Load();
 }
 
 void AsyncPolicyLoader::Init(const UpdateCallback& update_callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
@@ skipping 42 matching lines @@
   const base::TimeDelta age = now - last_modification_clock_;
   if (age < kSettleInterval) {
     *delay = kSettleInterval - age;
     return false;
   }
 
   return true;
 }
 
 }  // namespace policy