Split the policy refresh rate preference into user- and device-policy refresh rate.

chrome/browser/policy/configuration_policy_pref_store.cc

 // 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.
 
 #include "chrome/browser/policy/configuration_policy_pref_store.h"
 
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 228 matching lines @@
   { Value::TYPE_STRING, kPolicyAuthServerWhitelist,
     prefs::kAuthServerWhitelist },
   { Value::TYPE_STRING, kPolicyAuthNegotiateDelegateWhitelist,
     prefs::kAuthNegotiateDelegateWhitelist },
   { Value::TYPE_STRING, kPolicyGSSAPILibraryName,
     prefs::kGSSAPILibraryName },
   { Value::TYPE_BOOLEAN, kPolicyDisable3DAPIs,
     prefs::kDisable3DAPIs },
   { Value::TYPE_BOOLEAN, kPolicyDisablePluginFinder,
     prefs::kDisablePluginFinder },
   { Value::TYPE_INTEGER, kPolicyPolicyRefreshRate,

[Mattias Nissler (ping if slow), 2011/05/13 09:45:03]

We'll run into a problem here if we use the same policy type for both prefs:
Even if we have two different sources for policy values (user and device), we'll
merge them here, which is not what we want. Adding a new policy type goes
through the autogenerated code, but I'm not sure whether we want the device
policies in the JSON template. Need to discuss this with gfeher@ I guess

[sfeuz, 2011/05/17 14:27:42]

Added a policy for the DevicePolicyRefreshRate.

[Mattias Nissler (ping if slow), 2011/05/17 14:47:03]

Ah, but then there is the problem that the autogenerated policy decoder will
also accept that setting for user policy, which is not what we want. You may
want to look at the generator and talk to jkummerow/gfeher who have built the
code generation infrastructure.

-    prefs::kPolicyRefreshRate },
+    prefs::kDevicePolicyRefreshRate },
+  { Value::TYPE_INTEGER, kPolicyPolicyRefreshRate,
+    prefs::kUserPolicyRefreshRate },
   { Value::TYPE_BOOLEAN, kPolicyInstantEnabled, prefs::kInstantEnabled },
   { Value::TYPE_BOOLEAN, kPolicyDefaultBrowserSettingEnabled,
     prefs::kDefaultBrowserSettingEnabled },
   { Value::TYPE_BOOLEAN, kPolicyCloudPrintProxyEnabled,
     prefs::kCloudPrintProxyEnabled },
   { Value::TYPE_BOOLEAN, kPolicyTranslateEnabled, prefs::kEnableTranslate },
   { Value::TYPE_BOOLEAN, kPolicyBookmarkBarEnabled, prefs::kEnableBookmarkBar },
   { Value::TYPE_BOOLEAN, kPolicyAllowOutdatedPlugins,
     prefs::kPluginsAllowOutdated },
   { Value::TYPE_BOOLEAN, kPolicyAlwaysAuthorizePlugins,
@@ skipping 776 matching lines @@
   // Update the initialization flag.
   if (!initialization_complete_ &&
       provider_->IsInitializationComplete()) {
     initialization_complete_ = true;
     FOR_EACH_OBSERVER(PrefStore::Observer, observers_,
                       OnInitializationCompleted(true));
   }
 }
 
 }  // namespace policy