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

chrome/browser/policy/profile_policy_connector.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 <algorithm>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
 #include "chrome/browser/policy/cloud_policy_subsystem.h"
 #include "chrome/browser/policy/configuration_policy_pref_store.h"
 #include "chrome/browser/policy/profile_policy_connector.h"
 #include "chrome/browser/policy/user_policy_cache.h"
 #include "chrome/browser/policy/user_policy_identity_strategy.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
 
 namespace {
 
 const FilePath::CharType kPolicyDir[] = FILE_PATH_LITERAL("Device Management");
 const FilePath::CharType kTokenCacheFile[] = FILE_PATH_LITERAL("Token");
 const FilePath::CharType kPolicyCacheFile[] = FILE_PATH_LITERAL("Policy");
 
 const int64 kServiceInitializationStartupDelay = 2000;
 
 }  // namespace
@@ skipping 35 matching lines @@
 
 void ProfilePolicyConnector::ScheduleServiceInitialization(
     int64 delay_milliseconds) {
   if (cloud_policy_subsystem_.get())
     cloud_policy_subsystem_->ScheduleServiceInitialization(delay_milliseconds);
 }
 
 void ProfilePolicyConnector::Initialize() {
   if (identity_strategy_.get())
     identity_strategy_->LoadTokenCache();
-  if (cloud_policy_subsystem_.get())
-    cloud_policy_subsystem_->Initialize(profile_->GetPrefs(),
+  if (cloud_policy_subsystem_.get()) {
+    cloud_policy_subsystem_->Initialize(prefs::kUserPolicyRefreshRate,
                                         kServiceInitializationStartupDelay);
+    // Temporarily set the subsystem to listen to profile pref service, since
+    // we cannot yet serve user cloud policy to |local_state| and we don't want
+    // the profile reference in the subsystem.
+    profile_pref_registrar_.Init(profile_->GetPrefs());
+    profile_pref_registrar_.Add(prefs::kUserPolicyRefreshRate,
+                                cloud_policy_subsystem_.get());
+  }
 }
 
 void ProfilePolicyConnector::Shutdown() {
-  if (cloud_policy_subsystem_.get())
+  if (cloud_policy_subsystem_.get()) {
+    // Remove the temporary profile observer.
+    profile_pref_registrar_.RemoveAll();
     cloud_policy_subsystem_->Shutdown();
+  }
 }
 
 ConfigurationPolicyProvider*
     ProfilePolicyConnector::GetManagedCloudProvider() {
   return managed_cloud_provider_.get();
 }
 
 ConfigurationPolicyProvider*
     ProfilePolicyConnector::GetRecommendedCloudProvider() {
   return recommended_cloud_provider_.get();
@@ skipping 61 matching lines @@
     FOR_EACH_OBSERVER(ConfigurationPolicyProvider::Observer,
                       observer_list_, OnProviderGoingAway());
     browser_registrar_.reset();
     profile_registrar_.reset();
     browser_policy_provider_ = NULL;
     profile_policy_provider_ = NULL;
   }
 }
 
 }  // namespace policy