Handle policy refresh internally in ConfigurationPolicyPrefStore.

chrome/browser/policy/config_dir_policy_provider.cc

 // Copyright (c) 2010 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/config_dir_policy_provider.h"
 
 #include <set>
 
 #include "base/file_util.h"
 #include "base/values.h"
@@ skipping 51 matching lines @@
 
   // Enumerate the files and find the most recent modification timestamp.
   file_util::FileEnumerator file_enumerator(config_file_path(),
                                             false,
                                             file_util::FileEnumerator::FILES);
   for (FilePath config_file = file_enumerator.Next();
        !config_file.empty();
        config_file = file_enumerator.Next()) {
     if (file_util::GetFileInfo(config_file, &file_info) &&
         !file_info.is_directory) {
-      last_modification = std::min(last_modification, file_info.last_modified);
+      last_modification = std::max(last_modification, file_info.last_modified);

[danno, 2010/12/22 14:00:19]

unlrelated to this CL?

[Mattias Nissler (ping if slow), 2010/12/22 14:11:04]

Yes, but fixes the problem that we reload to often. I could create a separate CL
but was too lazy, and it's related to policy refresh.

     }
   }
 
   return last_modification;
 }
 
 ConfigDirPolicyProvider::ConfigDirPolicyProvider(
     const ConfigurationPolicyProvider::PolicyDefinitionList* policy_list,
     const FilePath& config_dir)
     : FileBasedPolicyProvider(
         policy_list,
         new ConfigDirPolicyProviderDelegate(config_dir)) {
 }
 
 }  // namespace policy